sitemap: the new /welcome routes are not excluded like the other journey pages #122

Stängd
öppnade 2026-07-31 18:46:01 +00:00 av supernaut · 1 kommentar
Ägare

astro.config.mjs filters journey and gated routes out of the sitemap — /signup, /account, /auth/* — but the two routes added for the sign-up journey were not added to that list.

/welcome and /en/welcome are public, static and harmless to index, so nothing breaks. But they are journey pages, meaningless out of context: a search result landing someone on "Start using Bitborg — sign in to finish setting up" with no account and no idea what preceded it is a poor entry point, and it competes with the pages we do want indexed.

Fix

Add both routes to the existing sitemap filter, following whatever pattern /signup uses there.

Done when

Neither /welcome nor /en/welcome appears in the generated sitemap.

`astro.config.mjs` filters journey and gated routes out of the sitemap — `/signup`, `/account`, `/auth/*` — but the two routes added for the sign-up journey were not added to that list. `/welcome` and `/en/welcome` are public, static and harmless to index, so nothing breaks. But they are journey pages, meaningless out of context: a search result landing someone on "Start using Bitborg — sign in to finish setting up" with no account and no idea what preceded it is a poor entry point, and it competes with the pages we do want indexed. ## Fix Add both routes to the existing sitemap filter, following whatever pattern `/signup` uses there. ## Done when Neither `/welcome` nor `/en/welcome` appears in the generated sitemap.
supernaut lade till detta till projektet Bitborg Web 2026-07-31 18:46:47 +00:00
Upphovsperson
Ägare

Already done — shipped by 8439fa4 (PR #125), whose body listed this issue but whose close keywords never fired.

Verified: the sitemap filter at `astro.config.mjs:120` excludes it —
`!//(signup|account|welcome)/?$/.test(page)` — and the live sitemap confirms it:
`curl https://www.gitborg.se/sitemap-0.xml | grep -c welcome` returns `0`.

Closing as already implemented; no code change was needed.

Already done — shipped by 8439fa4 (PR #125), whose body listed this issue but whose close keywords never fired. Verified: the sitemap filter at \`astro.config.mjs:120\` excludes it — \`!/\/(signup|account|welcome)\/?$/.test(page)\` — and the live sitemap confirms it: \`curl https://www.gitborg.se/sitemap-0.xml | grep -c welcome\` returns \`0\`. Closing as already implemented; no code change was needed.
Logga in för att delta i denna konversation.
Ingen milstolpe
Inget projekt
Inga tilldelade
1 deltagare
Notiser
Förfallodatum
Förfallodatumet är ogiltigt eller utanför gränserna. Använd formatet "åååå-mm-dd".

Inget förfallodatum satt.

Beroenden

Inga beroenden satta

Referens
bitborg/bitborg-web#122
Ingen beskrivning angiven.