[NO-TICKET] Call the JS/Node widget the Patchstack Connector - #96
Merged
Merged
Conversation
The JavaScript/Node install, troubleshooting and Pulse overview pages called the widget the "disclosure widget" or "Patchstack widget", and used "the connector" for the @patchstack/connect package. The widget is now the Patchstack Connector and the package is Connect. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Contributor
Author
|
/review |
|
Docs consistently rename the disclosure widget to Patchstack Connector for clarity. 🎯 Quality: 89% Excellent · 📦 Size: Small 📈 This month: Your 142nd PR — above team average · Averaging Excellent |
✅ Deploy Preview for deluxe-meerkat-8daf24 ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
daniloradovic
approved these changes
Sep 25, 2026
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
Three pages now call the site widget the Patchstack Connector, and call the
@patchstack/connectpackage Connect:Why
The app, the widget and
@patchstack/connectnow call the widget the Patchstack Connector (patchstack/connect#283, patchstack/sass-webvdp-widget#159, patchstack/saas#1870). The old name, "disclosure widget", names the public report form, which is off until the site is claimed. These pages also used "the connector" for the npm package. Keeping that next to the new widget name would make one word mean two things on the same page.Fix
Verified
npm run build: 197 pages built with no errors. The build rewrotepublic/schemas/threat-intel-npm.postman_collection.json; that change was reverted and is not part of this PR.Out of scope, worth a follow-up
The Patchstack Plugin → Patchstack Connector section already uses that name for the Drupal connector (and the former Joomla one). This PR leaves it alone, but the two products now share a name in the sidebar and in search. That needs a product decision before this merges.
🤖 Generated with Claude Code