Skip to content
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(organizations): use mantine in member actions dropdown TASK -1382 #5441

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

jamesrkiger
Copy link
Contributor

@jamesrkiger jamesrkiger commented Jan 23, 2025

🗒️ Checklist

  1. run linter locally
  2. update all related docs (API, README, inline, etc.), if any
  3. draft PR with a title <type>(<scope>)<!>: <title> TASK-1234
  4. tag PR: at least frontend or backend unless it's global
  5. fill in the template below and delete template comments
  6. review thyself: read the diff and repro the preview as written
  7. open PR & confirm that CI passes
  8. request reviewers, if needed
  9. delete this section before merging

📣 Summary

Replaces older custom components in the MemberActionsDropdown with Mantine menu.

👀 Preview steps

  1. As an admin or Owner of an MMO with a few members, navigate to the organization members page.
  2. Compare the actions dropdown between this branch and main. (note: We stuck with Mantine's default padding for the menu.)
  3. On PR branch, remove a member to confirm menu actually works.

@jamesrkiger jamesrkiger self-assigned this Jan 23, 2025
Copy link

@jamesrkiger jamesrkiger requested review from pauloamorimbr and removed request for magicznyleszek January 23, 2025 14:10
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