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

Check if a user belongs to a connected group #1130

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

Conversation

zak39
Copy link
Collaborator

@zak39 zak39 commented Feb 3, 2025

In this fix/refactor, I determine if a user is part of a connected group by following these steps:

  1. Check if the user is in the connected group.
  2. Check if the user is hard registered in the user group (SPACE-U).

The isUserConnectedGroup function returns true if the user is in a connected group, and false otherwise.

In the addGroup function, I check if the current user is in a connected group. If not, the user is not added to the group.

OP#3548

In this fix/refactor, I determine if a user is part of a connected group by following these steps:

1. Check if the user is in the connected group.
2. Check if the user is hard registered in the user group (SPACE-U).

The `isUserConnectedGroup` function returns true if the user is in a connected group, and false otherwise.

In the `addGroup` function, I check if the current user is in a connected group. If not, the user is not added to the group.
@zak39 zak39 added backend Modification related to backend debug Ongoing correction refactor labels Feb 3, 2025
@zak39 zak39 added this to the v5.0.0 milestone Feb 3, 2025
@zak39 zak39 self-assigned this Feb 3, 2025
@zak39
Copy link
Collaborator Author

zak39 commented Feb 3, 2025

I have to backport this PR for the v4.0.1

zak39 added 2 commits February 3, 2025 16:46
Mock the GroupFoldersGroupsMapper.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend Modification related to backend debug Ongoing correction refactor
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant