Make WordPress Core

Opened 6 months ago

Closed 5 months ago

Last modified 5 months ago

#61137 closed enhancement (fixed)

Allow view access of template rest endpoint to anyone with the edit_post capability

Reported by: fabiankaegy's profile fabiankaegy Owned by: fabiankaegy's profile fabiankaegy
Milestone: 6.6 Priority: normal
Severity: normal Version:
Component: REST API Keywords: has-patch has-unit-tests gutenberg-merge commit
Focuses: Cc:

Description

Sync ticket for https://github.com/WordPress/gutenberg/pull/60317

In order to render the block template in the locked template preview inside the post editor we need to be able to fetch the contents of any block templates/template parts for any user role that can edit a post.

Change History (12)

This ticket was mentioned in PR #6494 on WordPress/wordpress-develop by @fabiankaegy.


6 months ago
#1

  • Keywords has-patch added

#2 @swissspidy
6 months ago

  • Keywords needs-unit-tests added

#3 @fabiankaegy
6 months ago

  • Keywords has-unit-tests added; needs-unit-tests removed

I added some unit test cases that cover the changes in the flow

#4 @fabiankaegy
6 months ago

  • Keywords gutenberg-merge added

#5 @fabiankaegy
6 months ago

@youknowriad and @TimothyBlynJacobs since you worked on this one with me / reviewed in Gutenberg, would you be able to help get it into core also? We have 1 week till Beta 1 is supposed to be released and I want to make sure we have time for any remediation in case your review wants to change anything.

Thanks in advance :)

@youknowriad commented on PR #6494:


5 months ago
#6

I'll add this to my commit list if no one beats me to it.

#7 @youknowriad
5 months ago

  • Keywords commit added

@youknowriad commented on PR #6494:


5 months ago
#8

I have committed this PR (you can see the commit in Github on trunk) but there might be an issue currently in Trac preventing the changeset link from working https://core.trac.wordpress.org/changeset/58227

#9 @youknowriad
5 months ago

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

#10 @swissspidy
5 months ago

In 58229:

REST API: Remove default text domain from translatable strings after [58227].

See #61137.

#12 @gziolo
5 months ago

In 58232:

Tests: Update the generated fixture for REST API

Follow-up [58227].
See #61137.

Note: See TracTickets for help on using tickets.