Repository navigation
Revert "docs: add new-navigation-banner to discover-snyk classic pages (DOCT-2626 Phase B)" - #1595
Merged
Merged
Conversation
✅ Snyk checks have passed. No issues have been found so far.
💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse. |
VeronicaSnyk
approved these changes
Aug 4, 2026
Merged
3 tasks
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.
Summary
Reverts #1579 (merged earlier today) which inserted the reusable new-navigation-banner include into 39 discover-snyk pages with
nav_context: classic.Why
The banner links to
https://docs.snyk.io/getting-started/navigating-snyk, which currently returns 404 on live docs — the "Navigating Snyk" page for DOCT-2626 is still an unmerged GitBook draft (CR #28). Shipping the banner before the target lands sends users from real docs to a dead link.Rolling back until sequencing is right:
Phase A (#1577) is untouched — the include source files at
<section>/.gitbook/includes/new-navigation-banner.mdremain on main. They're inert without callers.Test plan
git diff main..revert/doct-2626-discover-snyk-rolloutshows only removals of the{% include %}line across the same 39 discover-snyk files (+78/-0in the original PR, now-78/+0in this revert).docs.snyk.io/readme.mdcontinues to show no banner (no visible change to end users, since the banner wasn't rendering yet or would have pointed at 404).Note
Low Risk
Documentation-only removal of a GitBook include; no product code or auth paths change.
Overview
This PR reverts the DOCT-2626 Phase B rollout on discover-snyk by deleting the
{% include %}line fornew-navigation-banner.mdfrom 39 GitBook pages that usenav_context: classic(homepage, getting started, pilot guide, implementation guides, start scanning, Snyk Learn, and supported languages).The banner pointed readers to Navigating Snyk at
https://docs.snyk.io/getting-started/navigating-snyk, which is not live yet, so showing it would send users to a 404. Phase A include files under.gitbook/includes/stay on main; they are unused until includes are added again after the target page ships.Reviewed by Cursor Bugbot for commit 5be0178. Bugbot is set up for automated code reviews on this repo. Configure here.