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

Rename role for consistency (leftover from change in v3) #10871

Merged
merged 1 commit into from
Jan 14, 2025

Conversation

mmattel
Copy link
Contributor

@mmattel mmattel commented Jan 13, 2025

References: #10870 (Transition from 'Guest' user to 'User Light' was incomplete in terms of translatable strings)

This PR does most likely not fix the issue completely, but contributes to make things consistable and understandable.

The change was defined in: https://github.com/owncloud/ocis/blob/9e1fe81069b526d76ed4da5d8044cb03ca3e1a4c/changelog/3.0.0_2023-06-06/rename-guest-role.md?plain=1

Note that the UUID's of the roles are the same.

  • Changing:
    roleId = settingsService.BundleUUIDRoleGuest --> roleId = settingsService.BundleUUIDRoleUserLight
  • Replacing
    BundleUUIDRoleGuest with BundleUUIDRoleUserLight in services/settings/pkg/service/v0/settings.go
  • Adding comments

@mmattel mmattel requested review from 2403905 and kobergj January 13, 2025 15:19
@mmattel mmattel requested a review from LukasHirt as a code owner January 13, 2025 15:19
@mmattel mmattel force-pushed the rename_bunlde_for_consistency branch from 819f29e to 4aab3a2 Compare January 13, 2025 15:22
@owncloud owncloud deleted a comment from update-docs bot Jan 13, 2025
@2403905 2403905 merged commit ce1cf9c into master Jan 14, 2025
4 checks passed
@mmattel mmattel deleted the rename_bunlde_for_consistency branch January 14, 2025 08:48
ownclouders pushed a commit that referenced this pull request Jan 14, 2025
Rename role for consistency (leftover from change in v3)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants