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
Add support for default role (#2631) #2676
Conversation
Schema Change Detected. Needs ingestion-core version bump Please run |
a33cb34
to
5eff148
Compare
The Java checkstyle failed. Please run You can install the pre-commit hooks with |
5eff148
to
a184950
Compare
The Java checkstyle failed. Please run You can install the pre-commit hooks with |
89c447a
to
7562983
Compare
ea6596d
to
53ba34d
Compare
flaky test from webhook
|
53ba34d
to
8437e97
Compare
@mithmatt can you rebase this PR we can merge it in |
- Add default field to role - Amend GET /roles to query default roles - Assign default role to new users being created
8437e97
to
06af66b
Compare
[open-metadata-ingestion] Kudos, SonarCloud Quality Gate passed! |
[catalog] Kudos, SonarCloud Quality Gate passed! |
Describe your changes :
When a new user is created, the default role is assigned to the user automatically by the backend.
Default role can be queried through GET /roles API
There is one more part remaining -- to switch default role from existing one to another.
See #2460 for details
Type of change :
Frontend Preview (Screenshots) :
For frontend related change, please link screenshots of your changes preview! Optional for backend related changes.
Checklist:
Reviewers