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

Inviting existing users fails #2025

Merged
merged 2 commits into from
Oct 13, 2023
Merged

Conversation

bkiahstroud
Copy link
Collaborator

Ref

Story

Trying to add a user active in other repositories to another tenant does not return any error but does not add the user.

Changes proposed in this pull request:

  • Always add invited users to the tenant's Registered Users group

Acceptance Criteria

  • Invited users are always added to the Registered Users group

Testing Instructions

  1. Login as a super admin
  2. Create a new tenant (e.g. Tenant A)
  3. Navigate to Tenant A > Dashboard > Manage Users
  4. Invite a new user who does not already exist in the application (i.e. across all tenants, not just one)
  5. Verify that the new user gets added to the Registered Users group in Tenant A
  6. Create a new tenant (e.g. Tenant B)
  7. Navigate to Tenant B > Dashboard > Manage Users
  8. Invite the same user from Step 4. Do not give it any individual roles in the invite form
  9. Verify that the user gets added to the Registered Users group in Tenant B

Notes

The primary issue here is that invited users who already have an account (i.e. an instance of User in the database) were not being added to the tenant's Registered Users group. This meant that, if a user was invited with no specific roles, they simply wouldn't show up in the tenant at all.

@samvera/hyku-code-reviewers

Fixes a bug where users who were invited with no roles would not
show up in the users list at all
@bkiahstroud bkiahstroud added the patch-ver for release notes label Oct 13, 2023
@bkiahstroud bkiahstroud merged commit 4a2fded into main Oct 13, 2023
@bkiahstroud bkiahstroud deleted the inviting-existing-users-fails branch October 13, 2023 20:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
patch-ver for release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants