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
CODENVY-2164: Add search user method by email/username for admins #2191
Conversation
@ApiResponses({@ApiResponse(code = 200, message = "The response contains searching result"), | ||
@ApiResponse(code = 400, message = "Missed required parameters, parameters are not valid"), | ||
@ApiResponse(code = 500, message = "Internal server error occurred")}) | ||
public Response find(@ApiParam("User e-mail fragment") |
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.
Usually searching criteria tends to extend. It is better to pass DTO
to avoid method signature refactoring in the future.
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.
I guess that in this case it is not critical, because user does not provide to many fields to search by, but for me it's ok to use DTO class instead of query parameters
Why |
added changes log |
Is this API documented in swagger? |
@slemeur yes |
36f92fd
to
72b135c
Compare
What does this PR do?
Provides a method for administrators that allow to search users by email/name parts. New method is case insensitive and finds users with names/emails that contains given part.
method path:
api.endpoint/admin/user/find
What issues does this PR fix or reference?
#2164
Changelog
Add search user method by email/username for admins
Release Notes
Add new API to allow system admin to search users by names and emails