Skip to content

fix(ehr): fix harvest and tests so that the e2e test can pass reliably#7957

Open
alexwillingham wants to merge 3 commits into
developfrom
alex/fix-flaky-harvest-breaking-insurance-part
Open

fix(ehr): fix harvest and tests so that the e2e test can pass reliably#7957
alexwillingham wants to merge 3 commits into
developfrom
alex/fix-flaky-harvest-breaking-insurance-part

Conversation

@alexwillingham

Copy link
Copy Markdown
Contributor

No description provided.

@alexwillingham alexwillingham requested a review from imbenham June 4, 2026 16:01
// the Coverage.order field (1 = primary, 2 = secondary) so a stale/incomplete account.coverage
// list does not blank out the patient's insurance on read — and so the write path, which reads
// existing coverages through this same function, does not then drop or mis-prioritize them.
if (!existingCoverages.primary) {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Is this saying that coverages needn't be attached to the account to be the coverage of record?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I think it means that account.coverage is still the main event. It's just that the primaryness and secondaryness are not related to the account attachment.

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.

2 participants