-
Notifications
You must be signed in to change notification settings - Fork 24
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
Fixes #25999: Disabling user rest API token feature from api-authorizations plugin UI #775
base: branches/rudder/8.1
Are you sure you want to change the base?
Conversation
PR updated with a new commit |
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.
After further consideration, we wan to totally disable (not show) the pop-up when the feature is disabled (globally or for an user), as if the feature was not here.
PR updated with a new commit |
PR rebased |
d118a08
to
9d55a88
Compare
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.
Perfect !
This PR is not mergeable to upper versions. |
https://issues.rudder.io/issues/25999
UI for the feature switch in Normation/rudder#6050 : essentially we don't display the user API token menu in the api-authorizations plugin snippet.
We still have an API endpoint to pass the configuration value, also there is some refactoring and formatting applied to the Elm code which knows about the status by calling the endpoint.
In the related PR, the user management roles and default providers are also changed, so there is some changes in the user-management plugin