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
Give permission to see Allergies for Event to Responsible Group #3495
Give permission to see Allergies for Event to Responsible Group #3495
Conversation
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.
Would love to see some unit tests as well😁
713856a
to
6914518
Compare
6914518
to
2d48d51
Compare
Looks good to me :) |
Codecov ReportAll modified lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #3495 +/- ##
=======================================
Coverage 88.20% 88.21%
=======================================
Files 666 666
Lines 21069 21083 +14
=======================================
+ Hits 18584 18598 +14
Misses 2485 2485
☔ View full report in Codecov by Sentry. |
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💯
Resolves: ABA-589 |
Signed-off-by: Daniel Yang Hansen <daniel.yang.hansen@gmail.com>
2d48d51
to
efb0226
Compare
Signed-off-by: Daniel Yang Hansen <daniel.yang.hansen@gmail.com>
Signed-off-by: Daniel Yang Hansen <daniel.yang.hansen@gmail.com>
Signed-off-by: Daniel Yang Hansen <daniel.yang.hansen@gmail.com>
Tests run fine ( |
Backend checks if user is part of responsible group when delegating authorization to view allergies.
The corresponding serializer has also been updated to include the "ResponsibleGroup" field so that frontend also can check if user should be able to see the allergy tab
Resolves: ABA-589