-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Subscription fhirpath criteria #5975
Subscription fhirpath criteria #5975
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #5975 +/- ##
============================================
+ Coverage 83.39% 83.52% +0.12%
- Complexity 26927 27334 +407
============================================
Files 1681 1701 +20
Lines 103965 105797 +1832
Branches 13189 13355 +166
============================================
+ Hits 86702 88365 +1663
- Misses 11613 11723 +110
- Partials 5650 5709 +59 ☔ View full report in Codecov by Sentry. |
...r-jpaserver-subscription/src/main/java/ca/uhn/fhir/jpa/topic/SubscriptionTriggerMatcher.java
Outdated
Show resolved
Hide resolved
...r-jpaserver-subscription/src/main/java/ca/uhn/fhir/jpa/topic/SubscriptionTriggerMatcher.java
Outdated
Show resolved
Hide resolved
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.
This looks great! Please add a changelog, but I'd say it's good to go!
Changelog now added with 5ed27f0 |
…ndroid compatible
Head branch was pushed to by a user without write access
This PR adds the logical evaluation of fhirPathCriteria to SubscriptionTopics