Make WordPress Core

Opened 8 weeks ago

Closed 8 weeks ago

Last modified 8 weeks ago

#63023 closed enhancement (fixed)

Add the site reading options to the index

Reported by: mamaduka's profile Mamaduka Owned by: mamaduka's profile Mamaduka
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 @spacedmonkey
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 @Mamaduka
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 @Mamaduka
8 weeks ago

  • Owner set to Mamaduka
  • Resolution set to fixed
  • Status changed from assigned to closed

In 59880:

REST API: Add the site reading options to the index.

Exposes page_for_posts, page_on_front and show_on_front reading settings via REST API index.

Props mamaduka, audrasjb, spacedmonkey, timothyblynjacobs.
Fixes #63023.

Note: See TracTickets for help on using tickets.