-
Couldn't load subscription status.
- Fork 2.9k
Closes #19338 - Add in_list and exact support on all id and enum fields in GraphQL v2 #20680
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
base: feature
Are you sure you want to change the base?
Closes #19338 - Add in_list and exact support on all id and enum fields in GraphQL v2 #20680
Conversation
|
Not a maintainer or reviewer - just dropping a quick thought. Thanks for the PR! From my understanding, If I’m off base or there’s a use case for the string lookups here, please feel free to ignore. Appreciate the work on this! |
|
@pheus You could be right; I only grabbed |
…_list-after-v1-namespace-refactor
|
Marking this as blocked by #20679 temporarily |
Closes: #19338
Converts all
idfields and all enum-based fields toFilterLookups, which supportin_listandexactquerying.BaseObjectTypeFilterMixinandChangeLogFilterMixin(which are used by most if not all field types) to defineidusing aFilterLookupinstead ofUNSET.filters.py, all fields defined as a choice of enum values has similarly been redefined using aFilterLookup, with the same new lookup support.Note: The linter really does not like this kind of typing syntax, but I wasn't able to find a better way to do it:
Note: Depends on #20667