Filter pages with page model field

[dilonne]

How can one use a page model field besides ID to filter pages? Decorating a page model with @register_query_field allows custom query parameters to be used with the singular query but not with the plural query.

[zerolab]

Have you tried @register_paginated_query_field ?

[dilonne]

Let me give it a try

[dilonne]

@register_paginated_query_field only adds two more arguments(page and per page) to the plural query

[zerolab]

The documentation says:

You can add custom query parameters like so:

@register_paginated_query_field(
    "advert",
    "adverts",
    {
        "id": graphene.Int(),
        "url": graphene.String(),
    },
)

then use it in your query:

{
    # Get a specific advert
    advert(url: "some-unique-url") {
        url
        text
    }
}

so you should be able to replace url with you field in the definition

[dilonne]

"advert" is the singular query in this case so that should work, my question is how do you get to specify such on the plural query "adverts", suppose we had a non unique field 'category' field on the model Advert and we want to query all adverts with that particular category

@register_paginated_query_field(
    "advert",
    "adverts",
    {
        "id": graphene.Int(),
        "url": graphene.String(),
        "category": graphene.Int()
    },
)

adverts(category: 2){
      url
}

The above wont work with the plural query which is supposed to return a list, adverts won't recognise category as an argument

advert(category:  2){
    url
}

The above will recognize category as an argument but isnt for returning lists.

My question is how do we get plural queries to recognize custom query arguments

[dilonne]

Ended up overriding @register_query_field to allow the plural query to be able to use custom query params just like the singular query

[zerolab]

Hey @dilonne,

Can you share your overrides? I don't have lots of time this week, but keen to improve the developer experience, or documentation where possible

[flyte]

Sorry to do this twice in one day, but this is also still an issue in the latest version! I'm struggling to work out how to add filters to the plural query. @zerolab perhaps this should be reopened too?

[dopry]

@flyte @zerolab I think the underlying issue here may be that query_params isn't documented... looking at the code in register_singular_query_field it looks like the query params are passed to the filter function in the resolver. But in register_paginated_query_field ti doesn't seem to be, and this will likely break if search_query is set, since .search() handles filters differently...