Skip to content

feat: use room names in chat URLs#939

Open
hmans wants to merge 10 commits into
mainfrom
hmans/room-names-in-urls
Open

feat: use room names in chat URLs#939
hmans wants to merge 10 commits into
mainfrom
hmans/room-names-in-urls

Conversation

@hmans

@hmans hmans commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

Adds readable room URL suffixes while keeping room IDs as the only routing key.

  • Canonical channel room pages redirect to /chat/{server}/{roomId}-{roomName} after the room loads; DM rooms stay at /chat/{server}/{roomId}.
  • Preserves existing ID-based links for room navigation, message links, thread links, notifications, last-room memory, quick switcher, and room lists.
  • Removes the discarded name-route implementation: no /r/{roomName} route, no roomByName GraphQL resolver, and no historical room-name alias projection.
  • Keeps room names free to overlap room IDs or static chat page names because the suffix is never used for lookup.
  • Updates architecture/FDR docs plus backend, frontend unit, and Playwright coverage for ID routes, canonical suffix redirects, stale suffix redirects, and message-link behavior.

@hmans hmans force-pushed the hmans/room-names-in-urls branch from 359f592 to ec4bc6c Compare June 17, 2026 17:33
hmans added 3 commits June 17, 2026 19:36
* origin/main:
  feat: group room files by date (#937)
  test: cover recent reaction toolbar recents (#936)
  chore: pass commands through mise chatto task (#938)
  fix: represent deleted room members (#934)
  test(e2e): cover upload configuration seams (#933)
  chore(main): release 0.2.1 (#922)
  fix: tolerate stale room members (#932)
  test(frontend): narrow redundant message e2e coverage (#930)

# Conflicts:
#	frontend/src/lib/gql/graphql.ts
@hmans hmans enabled auto-merge (squash) June 17, 2026 18:20
@hmans hmans disabled auto-merge June 17, 2026 18:33
@hmans hmans marked this pull request as draft June 17, 2026 18:39
@hmans hmans marked this pull request as ready for review June 17, 2026 19:02
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.

1 participant