-
Notifications
You must be signed in to change notification settings - Fork 334
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(orgAdmin): org admins can archive teams #10022
Conversation
WalkthroughThe changes enhance the user interface and functionality within the team management components of the dashboard. An Changes
Sequence Diagram(s)sequenceDiagram
participant User
participant OrgTeamsRow
participant ArchiveTeamModal
participant Backend
User->>OrgTeamsRow: Clicks on Archive button
OrgTeamsRow->>ArchiveTeamModal: Opens modal with team details
User->>ArchiveTeamModal: Confirms archiving
ArchiveTeamModal->>Backend: Sends archive request
Backend-->>ArchiveTeamModal: Returns success
ArchiveTeamModal->>User: Closes modal
Assessment against linked issues
Recent review detailsConfiguration used: CodeRabbit UI Files selected for processing (1)
Additional comments not posted (5)
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (invoked as PR comments)
Additionally, you can add CodeRabbit Configuration File (
|
packages/client/modules/userDashboard/components/OrgTeams/OrgTeamsRow.tsx
Outdated
Show resolved
Hide resolved
packages/client/modules/userDashboard/components/Organization/OrgNav.tsx
Show resolved
Hide resolved
packages/client/modules/userDashboard/components/OrgTeams/ArchiveTeamModal.tsx
Outdated
Show resolved
Hide resolved
packages/client/modules/userDashboard/components/OrgTeams/ArchiveTeamModal.tsx
Outdated
Show resolved
Hide resolved
packages/client/modules/userDashboard/components/OrgTeams/OrgTeamsRow.tsx
Outdated
Show resolved
Hide resolved
packages/client/modules/userDashboard/components/OrgTeams/OrgTeamsRow.tsx
Outdated
Show resolved
Hide resolved
packages/client/modules/userDashboard/components/OrgTeams/OrgTeamsRow.tsx
Outdated
Show resolved
Hide resolved
packages/client/modules/userDashboard/components/OrgTeams/OrgTeamsRow.tsx
Outdated
Show resolved
Hide resolved
@tianrunhe I just noticed that there seem to have been some changes on master to restructure this view: |
Turns out there's some existing design on this from Enrique and there's some code Bartosz has already written. So, I just modify the code to reuse them. As a result, much less changes 🎉 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I like it except for the "Other's Team" tag. I would prefer a "Team lead" and "Member" tag on the others and no tag on the ones where they're not a member.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 1
Fixes #10011
Description
A couple of new feature & improvements in the organization view:
Demo
https://www.loom.com/share/a63ff49655614390a711912c51a3445c
Testing scenarios
Team member/team lead view
Org admin view
Final checklist
Summary by CodeRabbit
New Features
ArchiveTeamModal
for archiving teams within the user dashboard, enhancing team management capabilities.OrgTeamsRow
component to display team information based on user roles, including a newisOrgAdmin
prop for tailored interface interactions.OrgTeamMembers
component, providing additional options based on user permissions.Bug Fixes
OrgTeams
component for clearer user feedback regarding team visibility.Documentation
OrgNav
component for improved clarity and readability.