Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Fix Reason for Visit display bug #2835

Merged
merged 1 commit into from
Oct 30, 2024
Merged

Fix Reason for Visit display bug #2835

merged 1 commit into from
Oct 30, 2024

Conversation

angelathe
Copy link
Collaborator

@angelathe angelathe commented Oct 30, 2024 •
edited
Loading

PULL REQUEST

Summary

  • Update FHIR path for "Reason for Visit" section to be case-insensitive to capture the sections regardless of capitalization.

Related Issue

Fixes #2745

Acceptance Criteria

  • FHIR path accurately captures Reason for Visit sections regardless of section title capitalization
  • Bug eCRs from QA efforts have been fixed

Additional Information

Example bug eCRs from QA efforts:

Screenshot

Example: 2ff3119d-b933-4236-8746-53726e291449

Before:
image

After:
Screenshot 2024-10-30 at 09 22 17

@angelathe angelathe self-assigned this Oct 30, 2024
@angelathe angelathe marked this pull request as ready for review October 30, 2024 16:56
@angelathe angelathe requested a review from mcmcgrath13 October 30, 2024 16:56
BobanL
BobanL approved these changes Oct 30, 2024
View reviewed changes
Copy link
Collaborator

@BobanL BobanL left a comment

Choose a reason for hiding this comment

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

😮

@angelathe angelathe added this pull request to the merge queue Oct 30, 2024
Merged via the queue into main with commit 877818d Oct 30, 2024
11 checks passed
@angelathe angelathe deleted the angela/2745-bug-reason-for-visit branch October 30, 2024 17:47
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@BobanL BobanL BobanL approved these changes

@lina-roth lina-roth Awaiting requested review from lina-roth lina-roth is a code owner

@gordonfarrell gordonfarrell Awaiting requested review from gordonfarrell gordonfarrell is a code owner

@JNygaard-Skylight JNygaard-Skylight Awaiting requested review from JNygaard-Skylight JNygaard-Skylight is a code owner

@austin-hall-skylight austin-hall-skylight Awaiting requested review from austin-hall-skylight austin-hall-skylight is a code owner

@mcmcgrath13 mcmcgrath13 Awaiting requested review from mcmcgrath13 mcmcgrath13 is a code owner

Assignees

@angelathe angelathe

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

BUG: Reason for Visit is not displaying
2 participants
@angelathe @BobanL