You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
the codegen plugin currently does not provide the parameter names defined in mutations/queries. It would be really nice to have parameter names available as well in the generated DgsConstants object as you could provide the constant to DataFetchingEnvironments getArgument() function instead of a string to retrieve the raw input values. Then we would get additional compiler validation if the parameter name should ever change & we wouldn't have to rely on mere strings or own constants.
Little background info: We are relying on the raw input values to validate if some nullable input fields are actually set by the client or just by default due to general object mapping.
reacted with thumbs up emoji reacted with thumbs down emoji reacted with laugh emoji reacted with hooray emoji reacted with confused emoji reacted with heart emoji reacted with rocket emoji reacted with eyes emoji
Uh oh!
There was an error while loading. Please reload this page.
-
Hello everyone,
the codegen plugin currently does not provide the parameter names defined in mutations/queries. It would be really nice to have parameter names available as well in the generated
DgsConstantsobject as you could provide the constant to DataFetchingEnvironmentsgetArgument()function instead of a string to retrieve the raw input values. Then we would get additional compiler validation if the parameter name should ever change & we wouldn't have to rely on mere strings or own constants.Little background info: We are relying on the raw input values to validate if some nullable input fields are actually set by the client or just by default due to general object mapping.
I am thinking about something like this:
Could we add support for something like this? Or is there some way to retrieve the parameter names I am just unaware of?
Beta Was this translation helpful? Give feedback.
All reactions