chore(groups): retire the /groups index in favor of the profile Groups tab - #197
Conversation
The standalone /groups listing is replaced by the profile Groups tab
(/{handle}?tab=groups), so the index URL no longer needs its own page.
- /groups now redirects to /home (no 404 for old links); the sub-routes
(/groups/create, /groups/import, /groups/[groupDid]) are unchanged.
- Drop /groups from the navbar root-path set (nothing links to it).
- Point create/import success + cancel redirects at /home (and the
remove-group redirect in org settings).
- Remove the obsolete leave-group dialog test and the /groups public-view
cases (old, removed functionality); keep the /endorsements coverage.
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
1 participant
Follow-up to #192. The standalone
/groupslisting is replaced by the profile Groups tab (/{handle}?tab=groups), so the index URL no longer needs its own page.Changes
/groupsnow redirects to/home(no 404 for old links). The sub-routes —/groups/create,/groups/import,/groups/[groupDid]— are unchanged./groupsfrom the navbar root-path set (nothing links to the index anymore)./home./groupspublic-view cases (old, removed functionality); kept/endorsementscoverage.Verification
npx tsc --noEmit,npm run typecheck:test, andnpm run lint(65 warnings / 0 errors, unchanged) all clean.vitest729/729 pass.🤖 Generated with Claude Code