new links#309
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (2)
✅ Files skipped from review due to trivial changes (2)
📝 WalkthroughWalkthroughThis PR updates several user-facing help/documentation URLs in the plugin: admin settings enhanced-measurement links, proxy bypass and page-exclusion help links, two admin module troubleshooting links, and an invalid-plugin-token Ajax error link, all pointing to new plausible.io docs anchors. ChangesAdmin Settings & Help Links
Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 inconclusive)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
There was a problem hiding this comment.
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
src/Admin/Settings/Page.php (1)
143-328:⚠️ Potential issue | 🟡 MinorUpdate remaining
plausible.io/wordpress-analytics-plugin#...doc links todocs/wordpress-integrationThere are still leftover references to the old docs path (beyond the existing
src/Admin/Module.phpentries):
src/Ajax.php:https://plausible.io/wordpress-analytics-plugin#oops-the-token-you-used-is-invalidsrc/Admin/Module.php:https://plausible.io/wordpress-analytics-plugin#if-the-proxy-script-is-slowsrc/Admin/Module.php:https://plausible.io/wordpress-analytics-plugin#stats-not-being-recorded-after-enabling-the-proxyreadme.txt:https://plausible.io/wordpress-analytics-plugin(setup link)Align these to the new
https://plausible.io/docs/wordpress-integration#...anchors used elsewhere (e.g., insrc/Admin/Settings/Page.php) for consistency.🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@src/Admin/Settings/Page.php` around lines 143 - 328, Update the remaining outdated plausible.io doc links to the new docs path: in class Ajax (src/Ajax.php) replace "https://plausible.io/wordpress-analytics-plugin#oops-the-token-you-used-is-invalid" with "https://plausible.io/docs/wordpress-integration#oops-the-token-you-used-is-invalid"; in class Module (src/Admin/Module.php) replace both "https://plausible.io/wordpress-analytics-plugin#if-the-proxy-script-is-slow" and "https://plausible.io/wordpress-analytics-plugin#stats-not-being-recorded-after-enabling-the-proxy" with "https://plausible.io/docs/wordpress-integration#if-the-proxy-script-is-slow" and "https://plausible.io/docs/wordpress-integration#stats-not-being-recorded-after-enabling-the-proxy" respectively; and in readme.txt replace "https://plausible.io/wordpress-analytics-plugin" (setup link) with "https://plausible.io/docs/wordpress-integration". Ensure exact anchor fragments are preserved when constructing the new URLs.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Outside diff comments:
In `@src/Admin/Settings/Page.php`:
- Around line 143-328: Update the remaining outdated plausible.io doc links to
the new docs path: in class Ajax (src/Ajax.php) replace
"https://plausible.io/wordpress-analytics-plugin#oops-the-token-you-used-is-invalid"
with
"https://plausible.io/docs/wordpress-integration#oops-the-token-you-used-is-invalid";
in class Module (src/Admin/Module.php) replace both
"https://plausible.io/wordpress-analytics-plugin#if-the-proxy-script-is-slow"
and
"https://plausible.io/wordpress-analytics-plugin#stats-not-being-recorded-after-enabling-the-proxy"
with
"https://plausible.io/docs/wordpress-integration#if-the-proxy-script-is-slow"
and
"https://plausible.io/docs/wordpress-integration#stats-not-being-recorded-after-enabling-the-proxy"
respectively; and in readme.txt replace
"https://plausible.io/wordpress-analytics-plugin" (setup link) with
"https://plausible.io/docs/wordpress-integration". Ensure exact anchor fragments
are preserved when constructing the new URLs.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: 04736bd6-6909-4bde-872e-898ea57ae5cb
📒 Files selected for processing (1)
src/Admin/Settings/Page.php
|
@metmarkosaric, please also change any remaining links elsewhere in the code. There are still leftover references to the old docs path (beyond the existing src/Admin/Module.php entries): Align these to the new https://plausible.io/docs/wordpress-integration#... anchors used elsewhere (e.g., in src/Admin/Settings/Page.php) for consistency. |
|
@Dan0sz I've changed the other two files but readme.txt can stay as it is as that link will still stay valid |
@Dan0sz I'll split out some content from the main WP landing page to a dedicated WP docs page when the new plugin version is ready. these are the new links
Summary by CodeRabbit