Skip to content

feat: add member name to kommunity member removal modal#1319

Open
Ducica wants to merge 1 commit intoinveniosoftware:masterfrom
oarepo:contribution-display-user-in-user-removal-modal
Open

feat: add member name to kommunity member removal modal#1319
Ducica wants to merge 1 commit intoinveniosoftware:masterfrom
oarepo:contribution-display-user-in-user-removal-modal

Conversation

@Ducica
Copy link
Copy Markdown
Contributor

@Ducica Ducica commented Jul 22, 2025

❤️ Thank you for your contribution!

Description

Modal for removal of member from the community does not contain the name of the member you are removing. Sometimes if user missclicks, or they are not sure which member they clicked, they need to go back to the main view and check.

Before

image

After

image

Checklist

Ticks in all boxes and 🟢 on all GitHub actions status checks are required to merge:

Frontend

Reminder

By using GitHub, you have already agreed to the GitHub’s Terms of Service including that:

  1. You license your contribution under the same terms as the current repository’s license.
  2. You agree that you have the right to license your contribution under the current repository’s license.

@Ducica Ducica force-pushed the contribution-display-user-in-user-removal-modal branch from 23349ad to ab381e2 Compare January 30, 2026 12:14
@Ducica
Copy link
Copy Markdown
Contributor Author

Ducica commented Jan 30, 2026

@tmorrell @utnapischtim Hello, could you please let us know if you are interested in this PR? Thank you, Dusan (CESNET)

Copy link
Copy Markdown
Contributor

@tmorrell tmorrell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like an excellent addition. Needs one more review to be merged, ideally from someone with more js experience.

@mirekys mirekys requested a review from kpsherva March 2, 2026 15:32
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.

2 participants