-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
REST API: Add missing options to the settings endpoint #2893
Conversation
I'm not sure how to fix the remaining failure. Is there a command for regenerating API fixtures? |
@Mamaduka If you run the tests locally, but running |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me.
Thanks for the PR! This was merged to WP trunk in https://core.trac.wordpress.org/changeset/53588, but I noticed that the new options are the only ones in |
Thanks for catching that, @SergeyBiryukov 🙇 |
I've made a follow-up commit that switches the type to integer as per the comment above :) |
Thanks for the confirmation that it was OK to change the type :) Merged in https://core.trac.wordpress.org/changeset/53589. Backported to the 6.0 branch in https://core.trac.wordpress.org/changeset/53590. |
PR registers options for the settings endpoints. Unfortunately, we missed them during WP 6.0 backports - WordPress/gutenberg#38607 (comment).
Testing Instructions
wp.data.select('core').getEntityRecord( 'root', 'site' );
Trac ticket: https://core.trac.wordpress.org/ticket/56058
This Pull Request is for code review only. Please keep all other discussions in the Trac ticket. Do not merge this Pull Request. See GitHub Pull Requests for Code Review in the Core Handbook for more details.