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

Add an arrow in a field #900

Merged
merged 1 commit into from
Mar 26, 2024
Merged

Add an arrow in a field #900

merged 1 commit into from
Mar 26, 2024

Conversation

zak39
Copy link
Collaborator

@zak39 zak39 commented Oct 13, 2023

I added an arrow for the creating group field.

Screenshot
image

@dorianne-arawa & @Clementine46 : Can you confirm me if it's okay for you ?

I added an arrow for the creating group field.

Signed-off-by: Baptiste Fotia <[email protected]>
@zak39 zak39 added frontend Modification related to frontend chore Modification without impact on the code (description, readme, comment) labels Oct 13, 2023
@zak39 zak39 self-assigned this Oct 13, 2023
@Clementine46
Copy link
Member

Perfect! Thanks @zak39 :)

@zak39 zak39 merged commit 63aa571 into main Mar 26, 2024
6 checks passed
@zak39 zak39 deleted the style/add-arrow-create-group branch March 26, 2024 14:53
zak39 added a commit that referenced this pull request Mar 26, 2024
Add these Pull Requests :

- #915
- #900
- #893
- #888

Signed-off-by: Baptiste Fotia <[email protected]>
zak39 added a commit to zak39/workspace that referenced this pull request Apr 4, 2024
Add these Pull Requests :

- arawa#915
- arawa#900
- arawa#893
- arawa#888

Signed-off-by: Baptiste Fotia <[email protected]>

feat(service,share): Limit the search to members

Limit the search to members of the same group except for specific
groups.

I splited the code in 2 classes :

- GroupMembersOnlyChecker : To check if the checkbox for the "Allow apps
  to use the Share API" and check if groups are listed from the
multiselect.
- ShareMembersOnlyFilter : to get the members only the same group or
  excepted a few groups from the multiselect.

Signed-off-by: Baptiste Fotia <[email protected]>

fix(service): Check if the current user is present in groups excluded

I forget to check if the current user is present in groups exlcluded before to filter the users list.

Signed-off-by: Baptiste Fotia <[email protected]>

chore(): remove a comment

Signed-off-by: Baptiste Fotia <[email protected]>

fix(share): Return users with common groups

Return users with common groups. Except users belonging to groups listed in the drop-down list (`Settings > Share > Restrict users to only share with users in their groups > Ingore the following groups when cheking group membership`).

Signed-off-by: Baptiste Fotia <[email protected]>

style(php): Run composer cs:fix

Signed-off-by: Baptiste Fotia <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore Modification without impact on the code (description, readme, comment) frontend Modification related to frontend
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants