-
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
Add isPartner filter to GET /user/list endpoint #1830
Conversation
✔️ Deploy Preview for dev-bloom ready! 🔨 Explore the source changes: 0767aa0 🔍 Inspect the deploy log: https://app.netlify.com/sites/dev-bloom/deploys/6148a7387d9144000792e19e 😎 Browse the preview: https://deploy-preview-1830--dev-bloom.netlify.app |
✔️ Deploy Preview for dev-storybook-bloom ready! 🔨 Explore the source changes: 0767aa0 🔍 Inspect the deploy log: https://app.netlify.com/sites/dev-storybook-bloom/deploys/6148a73803999a0007a8205e 😎 Browse the preview: https://deploy-preview-1830--dev-storybook-bloom.netlify.app |
✔️ Deploy Preview for dev-partners-bloom ready! 🔨 Explore the source changes: 0767aa0 🔍 Inspect the deploy log: https://app.netlify.com/sites/dev-partners-bloom/deploys/6148a7386f85b70007eb9ba2 😎 Browse the preview: https://deploy-preview-1830--dev-partners-bloom.netlify.app |
699f256
to
e26ff23
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.
@pbn4 works as expected 👍
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.
@dominikx96 and @pbn4 , I think this looks good. The only thing I think we might want to add, and this is for @pbn4 , is that in the user list endpoint, we probably also want to filter the users based on the jurisdiction of the user making the request. So that if I'm an admin for Alameda and I'm looking at the user's table, I don't see users for other jurisdiction like San Jose. This could also be done in separate PR, if we don't want to add more to this PR.
* Add isPartner filter to GET /user/list endpoint * Fix pagination issue on items per page change * Filter users by isPartner * Fix code style issues with Prettier Co-authored-by: Dominik Barcikowski <dominik@airnauts.com> Co-authored-by: Lint Action <lint-action@samuelmeuli.com>
Pull Request Template
Issue
Addresses # (#1651)
Description
Adds
isPartner
filter to GET /user/list endpoint.Type of change
How Can This Be Tested/Reviewed?
Please describe the tests that you ran to verify your changes. Provide instructions so we can review. Please also list any relevant details for your test configuration
Checklist:
yarn generate:client
if I made backend changes