Skip to content

Fix dead welcome page link in first-activation notice - #62

Open
deepench wants to merge 1 commit into
developfrom
fix/dead-welcome-page-link
Open

deepench wants to merge 1 commit into
developfrom
fix/dead-welcome-page-link

Conversation

@deepench

Copy link
Copy Markdown

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 is radiate-options (add_theme_page() in class-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-options instead, 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.

Before (dead link) After (loads correctly)
Before: 403 error After: Radiate Options page

(screenshots pushed to the evidence-pr-15 branch for this PR only -- fine to delete once reviewed)

How to test the changes in this Pull Request:

  1. Activate Radiate on a fresh site (or delete the radiate_admin_notice_welcome option to bring the notice back).
  2. On the WordPress dashboard, click the "welcome page" link in the green activation notice.
  3. Confirm it loads the Radiate Options / "Welcome to Radiate!" screen instead of an access-denied error.

Types of changes:

  • Bug fix (non-breaking change which fixes an issue)
  • Enhancement (modification of the currently available functionality)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Other information:

  • Have you added an explanation of what your changes do and why you would like us to include them?
  • Have you successfully ran tests with your changes locally?
  • Have you updated the documentation accordingly?

Changelog entry

Fix - First-activation welcome notice no longer links to a dead admin page.

@github-actions

Copy link
Copy Markdown

QA suite — refused, no regression spec

This PR changes product source but adds no spec, so the suite was
refused before booting WordPress — running it just to report the same
thing at the end costs runner minutes for nothing. Run
/claudegrill:verify-fix locally and let write-spec add the guard
to this branch, then push again.

Source files changed with no matching spec
inc/admin/class-radiate-welcome-notice.php

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

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-welcome with radiate-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.

@deepench
deepench requested a review from subin-shk September 29, 2026 06:21
@deepench deepench self-assigned this Sep 29, 2026
@deepench deepench added the bug label Sep 29, 2026
@deepench

Copy link
Copy Markdown
Author

Built theme zip for this branch (fix/dead-welcome-page-link), if you want to test it directly without checking out the branch:

radiate-fix-dead-welcome-page-link.zip (built via npm run build)

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants