Closed
Conversation
…0323-174045 Update content submodule (18 commits)
Six content types had published content but no matching Next.js route — they fell through to the catch-all recipient page. The validate-links CI only checked against content directories, not actual routes. New routes: use-cases, withdraw, stories, pricing, supported-networks. ContentSEOFooter in marketing layout (below main footer). verify-content.ts replaces validate-links.ts with 9 verification passes. Content submodule bumped to include PRs #18-#21 (republish locales, MDX upgrades, template fixes).
feat: content routes, SEO footer & verify-content script
Pass 10 tracks the number of published pages and errors if it drops. Prevents accidental mass-unpublish like the 72-page incident from commit 79cb1cd. Baseline stored in .verify-content-baseline (330 pages). To intentionally lower the count: echo <new-count> > .verify-content-baseline
…gression feat: page count regression guard in verify-content
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
| <div className="flex flex-col gap-px overflow-hidden rounded-sm border border-n-1"> | ||
| {stories.map((story) => ( | ||
| <Link | ||
| key={story.slug} |
Check failure
Code scanning / CodeQL
Stored cross-site scripting
| {stories.map((story) => ( | ||
| <Link | ||
| key={story.slug} | ||
| href={`/${locale}/stories/${story.slug}`} |
Check failure
Code scanning / CodeQL
Stored cross-site scripting
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Includes 6 newly published help articles + 4 stories with MDX.
…-content chore: replace Notion FAQ link with help center + bump content
/{locale} URLs (e.g. /en, /es-419, /pt-br) had no page and fell through
to the [...recipient] catch-all, resulting in 404s.
This broke breadcrumb "Home" links on all content/SEO pages since they
point to /{locale}.
Adds temporary (302) redirects for all 5 locale roots to /.
Non-permanent so we can replace with proper localized landing pages later.
The deposit route uses two URL prefixes:
- /deposit/from-{slug} for exchanges (entity data)
- /deposit/via-{slug} for rails/networks (DEPOSIT_RAILS)
verify-content.ts was mapping ALL deposit content to from-{slug},
so rail pages like via-sepa were never validated and broken links
to rail pages were not caught.
Changes:
- Hoist exchangeSlugs to module scope for reuse
- Route generation: exchanges get from-, non-exchange deposit content gets via-
- Published content check: same from-/via- logic in urlPattern
fix: redirect locale roots (/{locale}) to / to fix 404s
…urls fix: verify-content handles from-/via- deposit URL prefixes
Adds stories, use-cases, withdraw, supported-networks, and pricing pages to the sitemap. Uses listPublishedSlugs() so new content is automatically included on next build. ~40 published pages were missing from the sitemap.
fix: add missing content types to sitemap
Fails CI if a published content type has routes but is missing from sitemap.ts. Checks both content directories and singleton pages (supported-networks, pricing). Prevents the situation where new content types get published but never appear in the sitemap for search engine indexing.
…rage feat: verify-content Pass 11 — sitemap coverage check
Contributor
Author
|
Superseded by new PR with conflicts resolved. |
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
2 participants
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.
Branch Sync
main is 6 commits ahead of dev.
Commits to sync:
• Update content submodule to latest main (18 commits)
• Merge pull request #1784 from peanutprotocol/auto/update-content-20260323-174045
• feat: add missing content routes, SEO footer, and verify-content script
• Merge pull request #1787 from peanutprotocol/feat/content-overhaul-main
• feat: add page count regression check to verify-content
• Merge pull request #1788 from peanutprotocol/fix/verify-page-count-regression
Auto-generated by Chip 🐿️