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 available elements not shown in add #350

Merged
merged 1 commit into from
Apr 30, 2024

Conversation

carma12
Copy link
Collaborator

@carma12 carma12 commented Apr 22, 2024

Not all the available elements are being shown in the dual selector when opening the 'Add' modal, preventing those to be
added. This is due to the limitation performed in the query when retrieving the elements that are not being assigned
to a given user.

This issue was affecting the following elements: User groups, Netgroups.

Fixes: #348

@carma12 carma12 added the bug Something isn't working label Apr 22, 2024
@carma12 carma12 self-assigned this Apr 22, 2024
@carma12 carma12 force-pushed the available-elements-not-showing-add branch 4 times, most recently from b191d2b to a89579e Compare April 24, 2024 15:01
Not all the available elements are being
shown in the dual selector when opening
the 'Add' modal, preventing those to be
added. This is due to the limitation
performed in the query when retrieving
the elements that are not being assigned
to a given user.

Fixes: freeipa#348
Signed-off-by: Carla Martinez <[email protected]>
@carma12 carma12 force-pushed the available-elements-not-showing-add branch from a89579e to 5a86f14 Compare April 26, 2024 14:09
@carma12 carma12 requested a review from miskopo April 30, 2024 12:08
@miskopo
Copy link
Member

miskopo commented Apr 30, 2024

Temporary change LGTM, however, I would like to ask you to provide a tracking issue, so it is not lost. Thank you!

@carma12
Copy link
Collaborator Author

carma12 commented Apr 30, 2024

Temporary change LGTM, however, I would like to ask you to provide a tracking issue, so it is not lost. Thank you!

Issue created to track the definitive changes to those tab sections: #358

@miskopo
Copy link
Member

miskopo commented Apr 30, 2024

Thank you, @carma12 , I am approving this PR. Happy merging!

@miskopo miskopo added the ack Pull Request approved, it can be merged label Apr 30, 2024
@carma12 carma12 merged commit 504ae54 into freeipa:main Apr 30, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ack Pull Request approved, it can be merged bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Is a member of] 'Add' modal doesn't display all the available elements
2 participants