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
Simplify default role allocation for users #3061
Conversation
All tests passed locally
|
@@ -383,43 +391,5 @@ private void setDefaultToFalse(Role role) { | |||
throw EntityNotFoundException.byMessage(CatalogExceptionMessage.entitiesNotFound(Entity.USER)); | |||
} | |||
} | |||
|
|||
private void updateUsers(EntityReference addRole, EntityReference removeRole) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No changes to tests, since the logic remains the same.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1 LGTM
[open-metadata-ingestion] Kudos, SonarCloud Quality Gate passed! |
Test on maven CI failed due to known flaky test
See #2961 |
Merging... |
Describe your changes :
I worked on this to simplify the code changes on how default role is allocated to users.
Type of change :
Frontend Preview (Screenshots) :
For frontend related change, please link screenshots of your changes preview! Optional for backend related changes.
Checklist:
Reviewers
Backend: @sureshms @harshach