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

UI is not updated when organization membership of current organization is removed from person #2345

Closed
niklasva82 opened this issue Nov 11, 2024 · 2 comments · Fixed by #2484
Assignees
Labels
🐜 bug Something isn't working 🚪 entry-level Good for newcomers 🐬 Medium Just a nice sized issue. 🎨 needs-design

Comments

@niklasva82
Copy link
Member

niklasva82 commented Nov 11, 2024

Steps to reproduce

  1. Go to "My Organization"
  2. Go to a person (e.g. search for Rosa Luxemburg)
  3. Scroll down to the bottom
  4. Under "Organizations" -> "My Organization" (ie currently active organization), click the trashcan to remove the person from the current organization.

Expected Behaviour

Indicate that the operation was succesful and that the person is no longer a member of the current organization.

Actual Behaviour

Nothing happens

Screenshots (if you have any)

image

@richardolsson richardolsson added 🐜 bug Something isn't working 🎨 needs-design 🐬 Medium Just a nice sized issue. labels Nov 13, 2024
@richardolsson richardolsson added the 🚪 entry-level Good for newcomers label Jan 19, 2025
@richardolsson
Copy link
Member

I propose that the action should open a confirm dialog, which clearly states that the person will no longer be available, and once confirmed the user is navigated to the /people section.

@MattettaM
Copy link
Collaborator

I can start working on this and go with Rikards proposed design

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐜 bug Something isn't working 🚪 entry-level Good for newcomers 🐬 Medium Just a nice sized issue. 🎨 needs-design
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants