Skip to content

[stable31] fix(settings): show group label instead of gid in accounts management - #53411

Merged
skjnldsv merged 3 commits into
stable31from
backport/53363/stable31
Jun 12, 2025
Merged

[stable31] fix(settings): show group label instead of gid in accounts management#53411
skjnldsv merged 3 commits into
stable31from
backport/53363/stable31

Conversation

@backportbot

@backportbotbackportbotBot commented Jun 10, 2025

Copy link
Copy Markdown

@backportbotbackportbotBot added bug 3. to review Waiting for reviews papercut Annoying recurring UX issue with possibly simple fix. feature: users and groups feature: settings labels Jun 10, 2025
@backportbotbackportbotBot added this to the Nextcloud 31.0.6 milestone Jun 10, 2025
@nextcloud-botnextcloud-bot mentioned this pull request Jun 10, 2025
11 tasks
.map(group => {
// Try to match with more extensive group data
const availableGroup = this.availableSubAdminGroups.find(g => g.id === group.id)
return availableGroup.name ?? group.name ?? group.id

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.

Suggested change
return availableGroup.name ?? group.name ?? group.id
return availableGroup?.name ?? group.name ?? group.id

.map(group => {
// Try to match with more extensive group data
const availableGroup = this.availableGroups.find(g => g.id === group.id)
return availableGroup.name ?? group.name ?? group.id

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.

Suggested change
return availableGroup.name ?? group.name ?? group.id
return availableGroup?.name ?? group.name ?? group.id

@skjnldsv

This comment was marked as resolved.

@skjnldsv
skjnldsvforce-pushed the backport/53363/stable31 branch from 77060dd to dc59cfcCompareJune 12, 2025 19:19
skjnldsvand others added 2 commits June 12, 2025 22:02
Signed-off-by: skjnldsv <skjnldsv@protonmail.com>
Co-authored-by: Ferdinand Thiessen <opensource@fthiessen.de>
Signed-off-by: John Molakvoæ <skjnldsv@users.noreply.github.com>
@skjnldsv

Copy link
Copy Markdown
Member

/compile

@skjnldsv
skjnldsvforce-pushed the backport/53363/stable31 branch from 461b1a9 to 3d2c00aCompareJune 12, 2025 20:02
Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
@skjnldsv
skjnldsv enabled auto-merge June 12, 2025 20:19
@skjnldsv
skjnldsv disabled auto-merge June 12, 2025 20:35
@skjnldsv
skjnldsv merged commit 2f3ceec into stable31Jun 12, 2025
@skjnldsv
skjnldsv deleted the backport/53363/stable31 branch June 12, 2025 20:35
@nextcloud-botnextcloud-bot mentioned this pull request Jul 3, 2025
6 tasks
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewWaiting for reviewsbugfeature: settingsfeature: users and groupspapercutAnnoying recurring UX issue with possibly simple fix.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@skjnldsv@AndyScherzinger@susnux@nextcloud-command