Skip to content
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 endpoints to accomodate crud actions for users #26

Merged
merged 6 commits into from
May 1, 2024
Merged

Conversation

montesmoci
Copy link
Collaborator

No description provided.

@montesmoci montesmoci requested a review from HootDunk April 17, 2024 21:24
@montesmoci montesmoci requested a review from jrw972 April 23, 2024 18:41
@montesmoci montesmoci marked this pull request as ready for review April 23, 2024 18:41
@montesmoci montesmoci changed the title wip - add endpoint to accomodate crud actions for users Add endpoint to accomodate crud actions for users Apr 23, 2024
@montesmoci montesmoci changed the title Add endpoint to accomodate crud actions for users Add endpoints to accomodate crud actions for users Apr 23, 2024
.toList());
}

@Get("/tenants/{id}/users")
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Who is allowed to call this endpoint?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Only logged in users. Should we limit based on role?

@montesmoci montesmoci merged commit e16aa6f into main May 1, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants