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

Release/v1.1.2 #1682

Merged
merged 8 commits into from Jan 12, 2021
Merged

Release/v1.1.2 #1682

merged 8 commits into from Jan 12, 2021

Conversation

jasonbahl
Copy link
Collaborator

Release Notes

Bugfixes

  • (Bug/#1675 menu item ids inconsistent #1676) MenuItems are stored in the post table and should use the post loader. To use the post loader properly, the menu item IDs should be encoded as post:$id but were encoded as nav_menu_item:$id which breaks the principles of the Relay spec. This fixes that. MenuItems now output ids encoded as post:$id

…perly with WPGraphQL should they be passed to a node query

- Update cases of menu item IDs to be `post:$id` instead of `nav_menu_item:$id`
- Update tests to test that both the old nav_menu_item:$id and post:$id work for nav menu item node queries to support previously issued IDs
## Bugfixes

- MenuItems are stored in the post table and should use the post loader. To use the post loader properly, the menu item IDs should be encoded as `post:$id` but were encoded as `nav_menu_item:$id` which breaks the principles of the Relay spec. This fixes that. MenuItems now output ids encoded as `post:$id`
@jasonbahl jasonbahl self-assigned this Jan 12, 2021
@jasonbahl jasonbahl merged commit 0efb64e into master Jan 12, 2021
@jasonbahl jasonbahl deleted the release/v1.1.2 branch March 5, 2022 00:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant