-
Notifications
You must be signed in to change notification settings - Fork 297
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
Facility Filter for GET Users API #785
Comments
We have another endpoint specifically for this already made. We can build this feature if that does not solve our problem. |
@vigneshhari Why are we using a different API for this? Isn't it against the standard pattern? Opinion on getting rid of the API and building the filter? |
@gigincg there's one for districts. I'm working on adding the facilities filter to the GET users API. |
@gigincg we had a separate API for this because of the way we store this data internally. The facility user relation is a many-to-many type relation, we can bring in facility filters here but it would not be computationally inefficient. if there is a strong use case for this API, we can start thinking about implementation. |
The text was updated successfully, but these errors were encountered: