[NO-TICKET] Explain why the report button is missing, and name every way to connect a site - #88
Merged
mariojgt merged 1 commit intoSep 21, 2026
Conversation
…ct a site Both JS/Node pages told readers to add data-report-form="true" to reveal the "Report a vulnerability" button. The widget does not read that attribute, so anyone who followed it changed nothing and still had no button. The real reason is claim state: while a site is unclaimed the widget shows the "Connect this website" panel instead of the report button, and the button appears once the site is connected. Both pages now say that, and the install page gains a section listing the three ways to connect a site, led by the widget panel. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
|
Clear documentation explaining missing report button and connection methods. 🎯 Quality: 95% Elite · 📦 Size: Small 📈 This month: Your 144th PR — above team average · Averaging Excellent |
2 tasks
✅ Deploy Preview for deluxe-meerkat-8daf24 ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Contributor
Author
|
/review |
daniloradovic
approved these changes
Sep 21, 2026
mariojgt
deleted the
mario/no-ticket-widget-visibility-and-claim-routes
branch
September 21, 2026 13:18
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What changed
The two JavaScript / Node.js pages now explain the real reason the Report a vulnerability button does not show up after an install, and the install page gains a Connecting the site to your account section that lists all three routes, led by the widget's own panel.
Why the old explanation was wrong
Both pages told readers to set
data-report-form="true"on the script tag to reveal the button. The widget does not read that attribute: it is absent from theauto_init_data_attributestable and from theinit()parameter list in the widget reference. Anyone who followed the instruction changed nothing, still had no button, and had been pointed away from the actual cause.The actual cause is claim state.
data-build-modeis on by default, and while a site is unclaimed the widget shows a one-time "Connect this website" panel instead of the report button. Connect the site and the button takes its place for good. The real opt-out isdata-build-mode="false", which skips onboarding and always shows the report form.Fix
installing-on-javascript-node-projects.mdnpx @patchstack/connect claim. Notes that an unconnected site stays claimable by anyone who loads the page, because the UUID is public and claiming is first-come.data-report-formparagraph is replaced by the claim-state behaviour, the production-build case, and#patchstack/?patchstackfor owners on a live site.setupparagraph points at the new section instead of naming the dashboard link as the only way in.troubleshooting-javascript-node-projects.mdxdata-build-mode="false"as the way to always show the form. Step 2 explains that a hidden panel on a published build is expected rather than a failed install.How it was verified
npm run build— 197 pages, no broken links or MDX errors. Both new anchors (#connecting-the-site-to-your-account,#the-disclosure-widget) resolve against their headings. Behaviour checked against the widget's own public reference rather than inferred from the connector.Out of scope, worth a follow-up
public/schemas/threat-intel-npm.postman_collection.jsonis stale: a build regenerates it withmatched_rangecontent that the committed copy is missing. Unrelated to this change, so I reverted it here — it wants its own commit.The connector printed the same wrong expectation, fixed in patchstack/connect#277.
🤖 Generated with Claude Code