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

Bug fixes and Missing functionality implementation for Enabling UMG in UST #852

Open
wants to merge 16 commits into
base: feature/sign-umg
Choose a base branch
from

Conversation

NishantVarshnay28
Copy link

@NishantVarshnay28 NishantVarshnay28 commented Jun 6, 2024

Summary

Currently, UST does not support UMG (user in multiple groups). To enable this feature, a PR exists: link. This PR includes bug fixes and missing functionality from the previously mentioned PR.

The fixes include:

  • Correcting the Group Admin logic when updating an existing user. Initially, if a user had group admin access to a group and later the group was removed from the group_admin list but still existed in the directory group, the group was being removed instead of just revoking the admin privileges.
  • Ensuring groups are created with the correct case as specified in the config file, instead of always in lowercase.
  • Fixing the issue where group creation failed if the group was part of a different directory group.
  • Resolving primary group issues when updating the user.
  • Adding conditional code to execute only when UMG is enabled.
  • Correcting typos.

Testing Steps

Fixes #xxx

@NishantVarshnay28 NishantVarshnay28 changed the title Group admin and account admin logic fix Bug fixes and Missing functionality implementation for Enabling UMG in UST Jun 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants