-
Notifications
You must be signed in to change notification settings - Fork 4.5k
Closed
Labels
Good First IssueAn issue that's suitable for someone looking to contribute for the first timeAn issue that's suitable for someone looking to contribute for the first time[Status] In ProgressTracking issues with work in progressTracking issues with work in progress[Type] Code QualityIssues or PRs that relate to code qualityIssues or PRs that relate to code quality
Description
What problem does this address?
Ref: #67199
Currently, the site editor references the p
GET parameter to determine which page to show. However, there are some URLs that use old parameters (postType
, path
, etc)
Example:
gutenberg/packages/edit-post/src/components/more-menu/manage-patterns-menu-item.js
Lines 16 to 18 in 1c698f8
const patternsUrl = addQueryArgs( 'site-editor.php', { | |
path: '/patterns', | |
} ); |
The URL is http://localhost:8888/wp-admin/site-editor.php?path=/patterns, and is redirected to http://localhost:8888/wp-admin/site-editor.php?p=/pattern.
What is your proposed solution?
It would be good to update these URLs to avoid unnecessary URL redirection.
Infinite-Null
Metadata
Metadata
Assignees
Labels
Good First IssueAn issue that's suitable for someone looking to contribute for the first timeAn issue that's suitable for someone looking to contribute for the first time[Status] In ProgressTracking issues with work in progressTracking issues with work in progress[Type] Code QualityIssues or PRs that relate to code qualityIssues or PRs that relate to code quality