Have you tested this mutation? (Could be done with the GraphQL playground
for instance, or hut(1).)
I would've expected "PUBLIC" to require double-quotes, but maybe I'm
wrong.
On 7/1/23 19:44, Simon Ser wrote:
> Have you tested this mutation? (Could be done with the GraphQL playground> for instance, or hut(1).)> > I would've expected "PUBLIC" to require double-quotes, but maybe I'm> wrong.
I did test it using the playground. It's a GraphQL enum, and apparently
that means you can't use quotes around it, and you get an error saying
"Enum cannot represent non-enum value" if you try.
On Saturday, July 1st, 2023 at 18:50, Taavi Väänänen <hi@taavi.wtf> wrote:
> On 7/1/23 19:44, Simon Ser wrote:> > > Have you tested this mutation? (Could be done with the GraphQL playground> > for instance, or hut(1).)> > > > I would've expected "PUBLIC" to require double-quotes, but maybe I'm> > wrong.> > I did test it using the playground. It's a GraphQL enum, and apparently> that means you can't use quotes around it, and you get an error saying> "Enum cannot represent non-enum value" if you try.
I see, makes sense!
The patch looks good to me.