Skip to content

Addressed minor user registry bugs#1651

Open
cberger8 wants to merge 5 commits intodevfrom
cb_ur_minor_fixes
Open

Addressed minor user registry bugs#1651
cberger8 wants to merge 5 commits intodevfrom
cb_ur_minor_fixes

Conversation

@cberger8
Copy link
Collaborator

Closes Issues #1638, #1639, #1640, #1641, #1643

Summary

Fixed several minor issues in user registry logic. Detailed changes below.

Important Changes

org.controller.js

  • Removed unnecessary string length check

baseOrgRepository.js

  • Fixed incorrect date field when converting from registry to legacy object

baseUserRepository.js

  • Fixed incorrect date field when converting from registry to legacy object
  • Added missing save call when removing user roles
  • Fixed incorrect function for converting legacy to registry object

utils.js

  • Fixed incorrect order of arguments to isAdmin function

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.

1 participant