-
Notifications
You must be signed in to change notification settings - Fork 331
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
Restrict Admin from unlinking users from other district #1157
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #1157 +/- ##
==========================================
+ Coverage 60.75% 60.91% +0.16%
==========================================
Files 210 210
Lines 11533 11535 +2
Branches 1644 1645 +1
==========================================
+ Hits 7007 7027 +20
+ Misses 4285 4259 -26
- Partials 241 249 +8 ☔ View full report in Codecov by Sentry. |
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
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.
LGTM
Co-authored-by: Aakash Singh <[email protected]>
@Pranshu1902 add test cases |
@sainak @vigneshhari please review and merge the PR. I have added tests cases |
@Pranshu1902 the test are not catching the changes, try to assert the error message response |
@sainak done |
Co-authored-by: Aakash Singh <[email protected]>
Fixes #1152
Ensure that the admin's district is same as that of the facility user's district being deleted and also able to home facilities only for users within same district.