Skip to content

fix: Update message parser fields pt.2#1500

Open
angelathe wants to merge 2 commits into
mainfrom
angela/1356-message-parser-fields
Open

fix: Update message parser fields pt.2#1500
angelathe wants to merge 2 commits into
mainfrom
angela/1356-message-parser-fields

Conversation

@angelathe
Copy link
Copy Markdown
Collaborator

PULL REQUEST

Summary

Update the following message parser fields in the extended schema:

  • Reason for visit
  • Specimen type
  • Specimen collection date

TBD: Remove disabilities column or add new table for disability questionnaire answers

Related Issue

Fixes #1356

Acceptance Criteria

Please copy the acceptance criteria from your ticket and paste it here for your reviewer(s)

Additional Information

Anything else the review team should know?

Checklist

  • If this code affects the other scrum team, have they been notified? (In Slack, as reviewers, etc.)

@angelathe angelathe self-assigned this Jun 4, 2026
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Jun 4, 2026

Codecov Report

❌ Patch coverage is 0% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 92.27%. Comparing base (51bea25) to head (64b32b1).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
containers/message-parser/app/utils.py 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1500      +/-   ##
==========================================
+ Coverage   85.91%   92.27%   +6.35%     
==========================================
  Files         259       49     -210     
  Lines       11710     3391    -8319     
  Branches     1356        0    -1356     
==========================================
- Hits        10061     3129    -6932     
+ Misses       1642      262    -1380     
+ Partials        7        0       -7     
Flag Coverage Δ
ecr-viewer ?
ingestion ?
message-parser 96.04% <0.00%> (+0.04%) ⬆️
orchestration 85.29% <ø> (ø)
trigger-code-reference ?

Flags with carried forward coverage won't be shown. Click here to find out more.

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@angelathe angelathe marked this pull request as ready for review June 4, 2026 17:43
@angelathe angelathe requested a review from a team as a code owner June 4, 2026 17:43
@angelathe angelathe requested review from austin-hall-skylight and vochaparro2 and removed request for a team June 4, 2026 17:43
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.

Fix message parser fields Pt. 2

2 participants