Skip to content

feat: move sdk teams#1420

Open
rohanchkrabrty wants to merge 2 commits intomove-sdk-membersfrom
move-sdk-teams
Open

feat: move sdk teams#1420
rohanchkrabrty wants to merge 2 commits intomove-sdk-membersfrom
move-sdk-teams

Conversation

@rohanchkrabrty
Copy link
Contributor

Summary

Restructure views/teams/ into list/ and details/ subfolders to better organize teams list and team detail page components.

Changes

  • Moved teams list page, columns, add-team dialog, and list styles into views/teams/list/
  • Moved team detail page, general tab, members tab, member columns, delete dialog, invite dialog, and detail styles into views/teams/details/
  • Split teams.module.css into list/teams.module.css (table styles) and details/team-detail.module.css (tab/container styles)
  • Updated root views/teams/index.ts barrel to re-export from list/ and details/
  • Converted useEffect-based form resets in dialogs to handleOpenChange pattern for cleaner state cleanup
  • Removed <span> wrappers from dropdown menu items in team columns

Technical Details

  • All relative import paths updated for the additional nesting level (../../../../../)
  • Existing thin wrappers in components/organization/teams/ remain unchanged — they import from the ~/react/views/teams barrel which continues to export the same public API
  • DeleteTeamDialog lives in details/ and is cross-imported by list/teams-list-page.tsx since it's used in both contexts

@vercel
Copy link

vercel bot commented Feb 25, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
frontier Ready Ready Preview, Comment Feb 25, 2026 8:56pm

@rohanchkrabrty rohanchkrabrty self-assigned this Feb 25, 2026
@rohanchkrabrty rohanchkrabrty requested review from paanSinghCoder, rohilsurana and rsbh and removed request for paanSinghCoder and rsbh February 25, 2026 20:53
@coderabbitai
Copy link

coderabbitai bot commented Feb 25, 2026

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Tip

Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs).
Share your feedback on Discord.


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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@rohanchkrabrty rohanchkrabrty changed the base branch from main to move-sdk-members February 25, 2026 20:54
@rohanchkrabrty rohanchkrabrty changed the title Move sdk teams feat: move sdk teams Feb 25, 2026
@rohanchkrabrty rohanchkrabrty added the Do not merge Label to indicate that the PR is not ready to be merged even though might be (or not) approvals. label Feb 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Do not merge Label to indicate that the PR is not ready to be merged even though might be (or not) approvals.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant