Conversation
QA suite — refused, no regression specThis PR changes product source but adds no spec, so the suite was Source files changed with no matching spec |
There was a problem hiding this comment.
Copilot review overview
🟢 Approval recommended
The reviewed change resolves the invalid link with no unresolved blocking issues.
Review effort: Lite
Findings: None
What changed in this PR
Fixes the first-activation welcome notice’s dead link by targeting the registered Radiate Options page.
Changes:
- Replaced
radiate-welcomewithradiate-options.
| File | Description |
|---|---|
inc/admin/class-radiate-welcome-notice.php |
Corrects the welcome notice destination URL. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
|
Built theme zip for this branch ( radiate-fix-dead-welcome-page-link.zip (built via Upload it via Appearance > Themes > Add New > Upload Theme on a test site. Same evidence branch as the before/after screenshots above -- fine to delete once reviewed. |
Fixes #15
Changes proposed in this Pull Request:
The first-activation welcome notice's "welcome page" link points to
themes.php?page=radiate-welcome, a page slug that's never registered anywhere in the theme -- the only registered admin page isradiate-options(add_theme_page()inclass-radiate-dashboard.php). So every brand-new Radiate user who clicks the theme's own primary onboarding link on their very first screen hits WordPress's generic "Sorry, you are not allowed to access this page" error.Fixed by pointing the link at
radiate-optionsinstead, which already renders a genuine "Welcome to Radiate!" panel with Get Started / Next Steps / Further Actions sections -- exactly the getting-started content the link was supposed to lead to.(screenshots pushed to the
evidence-pr-15branch for this PR only -- fine to delete once reviewed)How to test the changes in this Pull Request:
radiate_admin_notice_welcomeoption to bring the notice back).Types of changes:
Other information:
Changelog entry
Fix - First-activation welcome notice no longer links to a dead admin page.