Skip to content

[stable30] fix(accounts): enhance UX for groups assignment - #53247

Merged
susnux merged 4 commits into
stable30from
backport/52776/stable30
Jun 2, 2025
Merged

[stable30] fix(accounts): enhance UX for groups assignment#53247
susnux merged 4 commits into
stable30from
backport/52776/stable30

Conversation

@backportbot

@backportbotbackportbotBot commented Jun 2, 2025

Copy link
Copy Markdown

Backport of #52776

Warning, This backport's changes differ from the original and might be incomplete ⚠️

Todo

  • Review and resolve any conflicts
  • Remove all the empty commits

Learn more about backports at https://docs.nextcloud.com/server/stable/go.php?to=developer-backports.

- fix 'this.isAdmin' to be 'this.settings.isAdmin'
- compute 'availableGroups' instead of setting it once mounted
- populate store and 'availableGroups' from search requests
Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
- rename 'available*Groups' to 'selected*Groups'
- populate store and 'availableGroups' from search requests
Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
…er functionality for sub-admin groups
Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
@Antreesy

Copy link
Copy Markdown
Contributor

/compile amend

Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
@Antreesy
Antreesy marked this pull request as ready for review June 2, 2025 07:43

@AntreesyAntreesy left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested

@susnux
susnux merged commit de1811a into stable30Jun 2, 2025
@susnux
susnux deleted the backport/52776/stable30 branch June 2, 2025 11:01
@nextcloud-botnextcloud-bot mentioned this pull request Jun 4, 2025
11 tasks
@nextcloud-botnextcloud-bot mentioned this pull request Sep 18, 2025
@nextcloud-botnextcloud-bot mentioned this pull request Sep 25, 2025
2 tasks
@blizzzblizzz mentioned this pull request Oct 14, 2025
Sign up for freeto 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.

3 participants

@Antreesy@susnux@nextcloud-command