-
Notifications
You must be signed in to change notification settings - Fork 996
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
GEN-1234 - Clean up suggestions when a user is deleted #17988
Conversation
@@ -213,7 +214,8 @@ | |||
User.__name__: "/users", | |||
CreateUserRequest.__name__: "/users", | |||
AuthenticationMechanism.__name__: "/users/auth-mechanism", | |||
Bot.__name__: "/bots", # We won't allow bot creation from the client | |||
Bot.__name__: "/bots", | |||
CreateBot.__name__: "/bots", |
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.
useful for testing
@@ -415,6 +418,9 @@ public ResultList<Suggestion> listAfter(SuggestionFilter filter, int limit, Stri | |||
List<Suggestion> suggestions = getSuggestionList(jsons); | |||
String beforeCursor; | |||
String afterCursor = null; | |||
if (nullOrEmpty(suggestions)) { | |||
return new ResultList<>(suggestions, null, null, total); | |||
} | |||
beforeCursor = after == null ? null : suggestions.get(0).getUpdatedAt().toString(); |
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.
without the check we got index out of bounds
@@ -53,5 +53,5 @@ | |||
} | |||
}, | |||
"additionalProperties": false, | |||
"required": ["JWTToken", "JWTTokenExpiry"] | |||
"required": ["JWTTokenExpiry"] |
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.
the JWTToken is created in the backend, no need to pass it when creating a user
The Java checkstyle failed. Please run You can install the pre-commit hooks with |
Quality Gate passed for 'open-metadata-ingestion'Issues Measures |
Describe your changes:
Type of change:
Checklist:
Fixes <issue-number>: <short explanation>