Skip to content
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

[Bugfix] mdm pointcut missing link update #6296

Merged

Conversation

Jake-Gillberg
Copy link
Contributor

@Jake-Gillberg Jake-Gillberg commented Sep 16, 2024

When an MDM link is updated to be NO_MATCH, it is possible that other links will be updated as well.

All updated links should be reflected in the link event available to the MDM_POST_UPDATE_LINK pointcut.

Resolves #6297

Copy link

codecov bot commented Sep 16, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 83.53%. Comparing base (406db33) to head (24b0797).
Report is 53 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff              @@
##             master    #6296      +/-   ##
============================================
- Coverage     83.54%   83.53%   -0.02%     
- Complexity    27432    27628     +196     
============================================
  Files          1707     1720      +13     
  Lines        106185   106946     +761     
  Branches      13397    13453      +56     
============================================
+ Hits          88710    89333     +623     
- Misses        11750    11835      +85     
- Partials       5725     5778      +53     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link

@gav-ashdown-galileo gav-ashdown-galileo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved

@tadgh tadgh self-requested a review October 4, 2024 18:17
@tadgh tadgh enabled auto-merge (squash) October 4, 2024 18:18
@tadgh
Copy link
Collaborator

tadgh commented Oct 4, 2024

LGTM, will merge and add a changelog once it passes all tests. Thanks for the contribution it is very appreciated!

auto-merge was automatically disabled October 4, 2024 19:35

Head branch was pushed to by a user without write access

@Jake-Gillberg Jake-Gillberg force-pushed the bugfix/mdm-pointcut-missing-link-update branch from 24b0797 to 2db8985 Compare October 4, 2024 19:35
@Jake-Gillberg
Copy link
Contributor Author

Had an issue with the format-check, where the check failed because of a failure to comment on the PR, even though spotless reported success. Rebased on master, but now require workflow approval to re-run.

@tadgh
Copy link
Collaborator

tadgh commented Oct 4, 2024

No worries, re-running now.

@tadgh tadgh merged commit 32651e1 into hapifhir:master Oct 7, 2024
60 of 63 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

MDM_POST_UPDATE_LINK pointcut does not contain all links updated when a link is updated to be NO_MATCH
4 participants