AxeWP / wp-graphql-gravity-forms

GraphQL API for interacting with Gravity Forms.
GNU General Public License v3.0
165 stars 29 forks source link

fix!: Keep `PageField` with previous page data when filtering `formFields` by `pageNumber`. #422

Closed justlevine closed 5 months ago

justlevine commented 5 months ago

What

This PR changes the behavior of the pageNumber filter on Form Fields connection, so that GF_Field_Page is included in the results for the previous pageNumber.

This is a breaking change.

Why

As noted in https://github.com/AxeWP/wp-graphql-gravity-forms/issues/208#issuecomment-2043083723, the Page field includes the data required to create back/next buttons for the page number preceeding it. But because the pageNumber is is for the upcoming page, it misses the field field.

How

Testing Instructions

Additional Info

Checklist: