Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Camel case arguments #1182

Closed
bszema opened this issue May 17, 2024 · 2 comments
Closed

Camel case arguments #1182

bszema opened this issue May 17, 2024 · 2 comments

Comments

@bszema
Copy link

bszema commented May 17, 2024

i have sth like this articles(first: Int!, after: ID, isFeatured: Boolean): ArticleCollection!

in resolvers code i must have sth like this:
@staticmethod
def resolve_articles(
category: CategoryProxy,
info: GraphQLResolveInfo,
first: int,
after: str | None = None,
isFeatured: bool | None = None,
):

i Would prefere to have this as python case

desirable resolver should look like this

@staticmethod
def resolve_articles(
category: CategoryProxy,
info: GraphQLResolveInfo,
first: int,
after: str | None = None,
is_featured: bool | None = None,
):

@rafalp
Copy link
Contributor

rafalp commented May 17, 2024

Add convert_names_case=True to the make_executable_schema call to enable this.

https://ariadnegraphql.org/docs/case-conversion

@bszema
Copy link
Author

bszema commented May 21, 2024

thanks!

@bszema bszema closed this as completed May 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants