-
Notifications
You must be signed in to change notification settings - Fork 284
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
Filter out those facilities that are already linked to user #1588
Conversation
@Pranshu1902 fix the failed test |
Co-authored-by: Aakash Singh <[email protected]>
Co-authored-by: Aakash Singh <[email protected]>
@Pranshu1902 the test is still failing |
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## master #1588 +/- ##
==========================================
+ Coverage 59.44% 60.01% +0.57%
==========================================
Files 208 208
Lines 11278 11330 +52
Branches 1597 1608 +11
==========================================
+ Hits 6704 6800 +96
+ Misses 4372 4305 -67
- Partials 202 225 +23
☔ View full report in Codecov by Sentry. |
@sainak review the backend, the functionality looks good to me |
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.
@Pranshu1902 are you sure pre-commit lint is working fine in your local?
Check the lint CI
tested locally, looks good to me @vigneshhari can you review it |
Proposed Changes
Associated Issue
Fixes #1581
Merge Checklist
/docs
Only PR's with test cases included and passing lint and test pipelines will be reviewed
@coronasafe/care-backend-maintainers @coronasafe/care-backend-admins