-
Notifications
You must be signed in to change notification settings - Fork 4.7k
Change default context of the block editor settings API endpoint #31538
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
Conversation
|
Hey @gziolo 👋 I assigned you as a reviewer to double-check if this should be the default context if none is passed to the API endpoint. What do you think? |
|
Size Change: 0 B Total Size: 1.31 MB ℹ️ View Unchanged
|
I'm curious how would it work in practice? Would it try to collect all possible settings or only a shared subset of settings? What about filters that plugins and sites can apply? |
Actually, I thought of this when I was having issues with the I'll close this PR |
Description
Updates the default context for this endpoint to be
allinstead ofpost-editorHow has this been tested?
Test the endpoint
/wp-json/__experimental/wp-block-editor/v1/settingsAnd expect to return the default settings.
Types of changes
Small fix
Checklist:
*.native.jsfiles for terms that need renaming or removal).