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 discharge_reason filter for patient #1440

Merged
merged 1 commit into from
Jul 7, 2023
Merged

Conversation

Ashesh3
Copy link
Member

@Ashesh3 Ashesh3 commented Jul 6, 2023

Proposed Changes

  • Allow patient list to be filtered with last_consultation__discharge_reason

Associated Issue

@coronasafe/care-backend-maintainers @coronasafe/care-backend-admins

Copy link
Member

@rithviknishad rithviknishad left a comment

Choose a reason for hiding this comment

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

This filter would work only when is_active=False right?

@sainak
Copy link
Member

sainak commented Jul 6, 2023

This filter would work only when is_active=False right?

we have a is_active filter for it

@Ashesh3
Copy link
Member Author

Ashesh3 commented Jul 6, 2023

This filter would work only when is_active=False right?

Yep, active=False is being automatically applied on the frontend and sent along with this filter when this filter is selected.

@vigneshhari vigneshhari merged commit 5b9b57f into master Jul 7, 2023
5 checks passed
@vigneshhari vigneshhari deleted the discharge-reason-filter branch July 7, 2023 09:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants