Skip to content
This repository has been archived by the owner on Aug 21, 2024. It is now read-only.

Fixed a bug with project-permission granting. #8126

Merged
merged 1 commit into from
Jun 27, 2023

Conversation

barankyle
Copy link
Member

@barankyle barankyle commented Jun 23, 2023

Summary

UI for adding a user to a project said to enter Invite Code, but if given a user ID, the error message said invalid user ID or invite code. Updated project.create handler to parse either input parameter and determine if it's really the other, i.e. re-assigning data.inviteCode to data.userId if it detects a UUID on data.inviteCode.

Moved some constant regexes from InviteService.ts to a new IdConstants file in common/src/constants. Updated references to these constants.

References

closes #8123

Checklist

  • If this PR is still a WIP, convert to a draft
  • When this PR is ready, mark it as "Ready for review"
  • ensure all checks pass
  • Changes have been manually QA'd
  • Changes reviewed by at least 2 approved reviewer

QA Steps

List any additional steps required to QA the changes of this PR, as well as any supplemental images or videos.

UI for adding a user to a project said to enter Invite Code, but if given
a user ID, the error message said invalid user ID or invite code. Updated
project.create handler to parse either input parameter and determine if it's
really the other, i.e. re-assigning data.inviteCode to data.userId if it
detects a UUID on data.inviteCode.

Moved some constant regexes from InviteService.ts to a new IdConstants file
in common/src/constants. Updated references to these constants.
@HexaField HexaField added this pull request to the merge queue Jun 27, 2023
Merged via the queue into dev with commit 3d83f8b Jun 27, 2023
8 checks passed
@HexaField HexaField deleted the 8123-project-user-add-fix branch June 27, 2023 02:48
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug]: UserID error when adding user as project owner
2 participants