Settings first in menu + plugin row links#6
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughConstructor registers a plugin_action_links_ filter wired to OpenTrust_Admin::add_plugin_action_links(), which prepends "Settings" and external "Docs" links. The Settings submenu is registered with priority 0. The POT template is updated with new admin-facing strings. ChangesPlugin Action Links
Estimated code review effort🎯 2 (Simple) | ⏱️ ~8 minutes Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
Re-applies the two admin tweaks from #6 onto the post-rename admin class: Settings/Docs links on the Plugins row, and Settings submenu pinned to position 0.
Two small admin UX bits.
Settings now sits at the top of the OpenTrust submenu, above the CPTs. Top-level click still opens it (was already the case).
On the Plugins screen, the OpenTrust row now has Settings and Docs links next to Deactivate. Docs points to plugins.ettic.nl/docs/opentrust.
Summary by CodeRabbit
New Features
Localization