#63023 closed enhancement (fixed)
Add the site reading options to the index
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | 6.8 | Priority: | normal |
Severity: | normal | Version: | |
Component: | REST API | Keywords: | gutenberg-merge |
Focuses: | Cc: |
Description
PR exposes the site reading settings via the REST index, which allows low-capability users to preview and change templates.
Until #48885 is resolved, the only option for exposing readonly settings is the REST API index.
Gutenberg PR: https://github.com/WordPress/gutenberg/pull/69106.
Change History (6)
This ticket was mentioned in PR #8345 on WordPress/wordpress-develop by @Mamaduka.
8 weeks ago
#1
- Keywords has-patch has-unit-tests added
#2
@
8 weeks ago
- Keywords has-patch has-unit-tests removed
@Mamaduka Can you review https://github.com/WordPress/wordpress-develop/pull/8380? If so then we can just commit that.
@Mamaduka commented on PR #8345:
8 weeks ago
#3
Create the Trac ticket and update unit test files.
I don't think it should block merge. But yeah could be a cool include or follow up.
Let's follow-up on this.
#4
@
8 weeks ago
@spacedmonkey, I had a quick look today, and it looks really good. I'll try to do a proper review/testing this week.
Unfortunately, I think we still have to commit to this change. The JS code needs to be compatible with the last two versions of WordPress. Low-capability users won't be able to load the editor without reading settings.
#5
@
8 weeks ago
- Owner set to Mamaduka
- Resolution set to fixed
- Status changed from assigned to closed
In 59880:
Gutenberg PR: https://github.com/WordPress/gutenberg/pull/69106.
Trac ticket: https://core.trac.wordpress.org/ticket/63023