Skip to content

fix(frontend): remove duplicated streams/streams/[streamId] route and add redirect - #1157

Merged
ogazboiz merged 3 commits into
LabsCrypt:mainfrom
omoh5:fix/1084-remove-duplicate-stream-route
Aug 3, 2026
Merged

fix(frontend): remove duplicated streams/streams/[streamId] route and add redirect#1157
ogazboiz merged 3 commits into
LabsCrypt:mainfrom
omoh5:fix/1084-remove-duplicate-stream-route

Conversation

@omoh5

@omoh5 omoh5 commented Jul 30, 2026

Copy link
Copy Markdown
Contributor

Summary

Fixes #1084 — removes the dead /streams/streams/[streamId] route that was a leftover from a previous refactor, and adds a permanent redirect to the canonical /streams/:id route.

Problem

  • frontend/src/app/streams/streams/[streamId]/page.tsx was a duplicated dynamic route alongside frontend/src/app/streams/[id]/page.tsx
  • The code-searcher confirmed zero references to the streams/streams/ path anywhere in the codebase
  • With two competing dynamic routes, Next.js can route to either one, causing confusion for navigation and SEO (duplicate content at two URLs)

Changes

🔴 Removed

  • frontend/src/app/streams/streams/[streamId]/page.tsx (343 lines deleted)
    • Confirmed dead code — no imports, links, or navigation references point to it
    • All links in the codebase use /streams/<id> which resolves to the [id] route

🟢 Added

  • frontend/next.config.ts — permanent redirect (HTTP 308)
    • /streams/streams/:streamId/streams/:streamId
    • Ensures any external links or bookmarks to the old path seamlessly land on the correct page

Verification

  • npx tsc --noEmit — no new errors (pre-existing errors in useIncomingStreams.test.ts are unrelated)
  • Pre-commit lint hook passed
  • Code reviewer confirmed no issues
  • Directory confirmed removed

Out of scope

  • Restructuring the whole streams route tree
  • Changing the stream detail UI

omoh5 and others added 3 commits July 30, 2026 07:07
… add redirect

Delete the dead /streams/streams/:streamId route that was a leftover from
refactoring streamId to id. Add a permanent redirect in next.config.ts so
any external links to the old path land on the canonical stream details page.

Closes LabsCrypt#1084

@ogazboiz ogazboiz left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

verified the duplicate route really exists on main (frontend/src/app/streams/streams/[streamId]/page.tsx, a full stale copy of the details page) and the permanent redirect in next.config.ts is the right way to keep old links working. merging. if you want to keep contributing, join us on Telegram: https://t.me/+DOylgFv1jyJlNzM0

@ogazboiz
ogazboiz merged commit 9107757 into LabsCrypt:main Aug 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Frontend] frontend/src/app/streams/streams/[streamId]/page.tsx duplicated route nesting looks like a stray leftover from a refactor

2 participants