-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
User Permissions on TestSuite and TestCases #19072
Conversation
…nMetadata into issue-11002-v1
@@ -724,7 +725,8 @@ public Response delete( | |||
@Parameter(description = "FQN of the logical test suite", schema = @Schema(type = "String")) | |||
@PathParam("name") | |||
String name) { | |||
OperationContext operationContext = new OperationContext(entityType, MetadataOperation.DELETE); | |||
OperationContext operationContext = | |||
new OperationContext(Entity.TEST_SUITE, MetadataOperation.DELETE); |
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.
wouldn't entityType be the same as Entity.TEST_SUITE?
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.
LGTM
The Java checkstyle failed. Please run You can install the pre-commit hooks with |
Quality Gate passed for 'open-metadata-ui'Issues Measures |
Quality Gate passed for 'open-metadata-ingestion'Issues Measures |
Describe your changes:
Fixes #11002
Fixes #17948
I worked on ... because ...
Type of change:
Checklist:
Fixes <issue-number>: <short explanation>