-
Notifications
You must be signed in to change notification settings - Fork 382
Closed
Labels
ChangeloggedWhether the issue/PR has been added to release notes.Whether the issue/PR has been added to release notes.GroomedWS:UXWork stream for UX/Front-endWork stream for UX/Front-end
Milestone
Description
Bug Description
Originally reported in #2692. When AMP is not enabled for a post type, there should be no warning displayed users who do not the manage_options capability, since there is no action the user can take. Non-admin users also would likely not care about AMP in the first place.
Relates to #2673.
Expected Behaviour
No warning should be displayed.
Steps to reproduce
- Sign-in as a non-admin user.
- Disable AMP for a given post type.
- Edit an instance of the post type.
- See the warning.
- Click link and see access denied notice.
Screenshots
Block editor:
Classic editor:
Clicking link:
Additional context
- WordPress version:
- Plugin version:
- Gutenberg plugin version (if applicable):
- AMP plugin template mode:
- PHP version:
- OS:
- Browser: [e.g. chrome, safari]
- Device: [e.g. iPhone6]
Do not alter or remove anything below. The following sections will be managed by moderators only.
Acceptance criteria
Implementation brief
QA testing instructions
Demo
Changelog entry
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
ChangeloggedWhether the issue/PR has been added to release notes.Whether the issue/PR has been added to release notes.GroomedWS:UXWork stream for UX/Front-endWork stream for UX/Front-end


