Skip to content
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

Sync: Fix Appearance > Template Parts URL #4142

Closed

Conversation

Mamaduka
Copy link
Member

@@ -210,7 +210,7 @@
$submenu['themes.php'][6] = array(
__( 'Template Parts' ),
'edit_theme_options',
'site-editor.php?postType=wp_template_part',
'site-editor.php?postType=wp_template_part&path=/wp_template_part/all',
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If I remember correctly, there's no need to escape static strings.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No need to escape this string as it is not exposed to user edits, indeed.

@ntsekouras ntsekouras requested a review from gziolo February 28, 2023 12:26
Copy link
Contributor

@audrasjb audrasjb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

Copy link
Contributor

@hellofromtonya hellofromtonya left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • Confirmed the PR includes the change from Gutenberg PR 48343 ✅
  • Able to reproduce the issue ✅
  • Confirmed this patch fixes the issue ✅ (See Test Report)
  • Code is consistent with multiple args in this file ✅

Ready for commit 👍

@hellofromtonya
Copy link
Contributor

@Mamaduka Mamaduka deleted the fix/template-parts-mode-url branch May 9, 2023 09:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants