Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
Updates documentation to improve the accuracy/usability of links across Snap docs and component READMEs, aligning references with current repos/routes and making link behavior consistent in rendered docs.
Changes:
- Update/standardize documentation links (GitHub + docs site) across multiple markdown files.
- Expose previously commented “events methods” section in
snap-trackerREADME and point to the publicbeacon.jsdocs. - Extend
doc-app.jslink-rewrite rules to route additional GitHub links into the docs app.
Reviewed changes
Copilot reviewed 16 out of 17 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
| packages/snap-tracker/README.md | Updates beacon.js reference and adds events docs section/link. |
| packages/snap-preact/components/src/components/Trackers/ResultTracker/readme.md | Adds mention of withTracking alternative. |
| packages/snap-preact/components/src/components/Templates/RecommendationBundleVertical/readme.md | Converts Swiper links to explicit HTML anchors. |
| packages/snap-preact/components/src/components/Templates/RecommendationBundle/readme.md | Converts Swiper links to explicit HTML anchors; minor formatting changes. |
| packages/snap-preact/components/src/components/Templates/Recommendation/readme.md | Converts Swiper links to explicit HTML anchors; introduces a malformed anchor text. |
| packages/snap-preact/components/src/components/Molecules/FacetSlider/readme.md | Converts react-ranger link to explicit HTML anchor. |
| packages/snap-preact/components/src/components/Molecules/Carousel/readme.md | Converts Swiper links to explicit HTML anchors; minor formatting changes. |
| packages/snap-preact/components/src/components/Atoms/Price/readme.md | Updates toolbox docs link and uses HTML anchor. |
| packages/snap-preact/components/src/components/Atoms/FormattedNumber/readme.md | Updates toolbox docs link and uses HTML anchor. |
| images/branch-override.png | Adds/updates documentation image asset. |
| docs/TEMPLATES_ABOUT.md | Updates wording and component library link text. |
| docs/SNAP_RECOMMENDATIONS.md | Updates Storybook route links for Recommendation/Bundle docs. |
| docs/REFERENCE_CUSTOM_RECOMMENDATION_TEMPLATES.md | Updates Storybook route link for Recommendation template reference. |
| docs/REFERENCE_CONFIGURATION.md | Updates UrlTranslator config link path to directory route. |
| docs/BUILD_DEPLOY_CHECKLIST.md | Fixes Foreground Filters link target. |
| docs/BUILD_DEPLOY.md | Updates Vite preset link to npm package page. |
| doc-app.js | Adds additional GitHub→docs route rewrite mappings. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
packages/snap-preact/components/src/components/Templates/Recommendation/readme.md
Outdated
Show resolved
Hide resolved
packages/snap-preact/components/src/components/Molecules/Carousel/readme.md
Outdated
Show resolved
Hide resolved
packages/snap-preact/components/src/components/Molecules/FacetSlider/readme.md
Outdated
Show resolved
Hide resolved
packages/snap-preact/components/src/components/Templates/RecommendationBundle/readme.md
Outdated
Show resolved
Hide resolved
packages/snap-preact/components/src/components/Templates/Recommendation/readme.md
Outdated
Show resolved
Hide resolved
korgon
approved these changes
Mar 11, 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.
No description provided.