-
Notifications
You must be signed in to change notification settings - Fork 1.4k
Several IPS fixes #5297
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
Several IPS fixes #5297
Conversation
Formatting check succeeded! |
… into ja_20230909_ips_fixes
@jamesagnew Could a fix for this line be rolled in here as well? Thymeleaf error from AllergyIntolerances missing reaction.severity (e.g. https://fhir.ips-demo.dev.cirg.uw.edu/fhir/Patient/10953/$summary)
|
@daniellrgn yup - i actually caught that one in a sweep i did this afternoon for potential NPEs, it's fixed in this PR |
@daniellrgn at least i think it is ... if you wanted to test on hapi.fhir.org that'd be awesome. All fixes in this PR are deployed there as of now. |
Thank you @jamesagnew , that seems to have done the trick! |
Codecov ReportPatch coverage has no change and project coverage change:
Additional details and impacted files@@ Coverage Diff @@
## master #5297 +/- ##
============================================
+ Coverage 81.32% 82.79% +1.46%
- Complexity 23650 26385 +2735
============================================
Files 1425 1615 +190
Lines 86399 102596 +16197
Branches 11677 13116 +1439
============================================
+ Hits 70265 84942 +14677
- Misses 10947 12008 +1061
- Partials 5187 5646 +459 ☔ 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.
Looks good! Approved
No description provided.