[docs] Fix debugging /docs/next/ redirects following 0.76 cut #4297
Conversation
✅ Deploy Preview for react-native ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
|
Aren't these still available under next? |
|
@blakef I'd assume that under We may yet have the unwanted behaviour of lack of redirects for older docs versions. |
website/static/_redirects
Outdated
| /docs/next/architecture-overview /architecture/overview | ||
| /docs/next/fabric-renderer /architecture/fabric-renderer | ||
| /docs/next/render-pipeline /architecture/render-pipeline | ||
| /docs/next/xplat-implementation /architecture/xplat-implementation | ||
| /docs/next/view-flattening /architecture/view-flattening | ||
| /docs/next/threading-model /architecture/threading-model | ||
| /docs/next/architecture-glossary /architecture/glossary |
There was a problem hiding this comment.
You need to add new redirect: both /docs/next/xyz and /docs/xyz should redirect to /architecture/xyz
There was a problem hiding this comment.
Do we have any live references, elsewhere on the internet, to these old pages? IMO we don't need to be defensive with /next links, and this will only kick the can down the road.
Potentially, there's another case we have been missing up to this point, in the other direction. We may want to add redirects from e.g. /docs/0.75/architecture-overview (404) to /architecture/overview etc.
There was a problem hiding this comment.
But yeah, let me reduce this PR to debugging links only — will leave this with you.
huntie
force-pushed
the
0.76-cut-redirects
branch
from
60afd79
to
fbe801c
Compare
huntie
changed the title
Updates redirects following #4294. Removed routes under
/docs/nexthave now shipped under/docs.