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

791 #1575

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

791 #1575

wants to merge 12 commits into from

Conversation

myapos
Copy link
Contributor

@myapos myapos commented Mar 15, 2024

Description 📣

#791

Type ✨

  • [ X] New feature

This PR allows to give a user access to multiple projects at once. In order to do this I modified the organization access control page to fetch for each user the projects that already belongs to and added an action that will display a modal. The modal contains a table with all the available projects that a user can be added. The table also contains a search box and checkboxes to allow multiple selection of the users. I applied client side validation in order to avoid to send empty requests to the backend.

In the backend side, I added a new API call that will fetch all organization members with the projects that a user already belongs. Please let me know if this API call has to be added to the swagger too.

Further improvements could be

  • to add an action to remove project as well from the organization access control page
  • enhance the Table component with checkboxes and search ability for better reusability
  • add more automated tests

Tests 🛠️

empty_file_structure_fix-2024-03-15_19.17.44.mp4

2024-03-15_19-26
2024-03-15_19-26_1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants