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

fix: Unsearchable disabled users by their email or name-EXO-69348-Meeds-io/meeds#1827 #4127

Merged
merged 1 commit into from
Oct 22, 2024

Conversation

Jihed525
Copy link
Contributor

Before this change, when create user with a First name different than his login, set user as a disabled user, select disabled filter and search for user by his name or his email, user isn't displayed in the result list.
To resolve this problem, we make a condition in the inputUserFilter component to display the component only if the filter is enabled.
After this change, Hide filter by name or email field when disabled users list displayed on user management application.

…ds-io/meeds#1827.

Before this change, when create usera with a First name different than his login, set usera as a disabled user, select disabled filter and search for usera by his name or his email, usera isn't displayed in the result list.To resolve this problem, in the inputUserFilter component we make a condition to display the component only if the filter is enabled.After this change, Hide filter by name or email field when disabled users list displayed on user management application.

(cherry picked from commit 0711574)
@Jihed525 Jihed525 self-assigned this Oct 22, 2024
@Jihed525 Jihed525 enabled auto-merge (squash) October 22, 2024 08:25
@github-actions github-actions bot added the partialCIBuild Perform Partial CI Build label Oct 22, 2024
@Jihed525 Jihed525 merged commit b5c0478 into stable/6.5.x-exo Oct 22, 2024
5 of 6 checks passed
@Jihed525 Jihed525 deleted the backportSTABLE-69348 branch October 22, 2024 08:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
partialCIBuild Perform Partial CI Build
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants