Skip to content

refactor: restructure ECS README into user-focused deployment guide#1459

Draft
alismx wants to merge 8 commits into
mainfrom
alis/ecs
Draft

refactor: restructure ECS README into user-focused deployment guide#1459
alismx wants to merge 8 commits into
mainfrom
alis/ecs

Conversation

@alismx
Copy link
Copy Markdown
Collaborator

@alismx alismx commented May 10, 2026

No description provided.

Add JSON task definitions and README for deploying eCR Viewer services
to Amazon ECS, including ecr-viewer, fhir-converter, fhir-converter-proxy,
ingestion, message-parser, orchestration, and trigger-code-reference.
@alismx alismx requested a review from a team as a code owner May 10, 2026 01:37
@alismx alismx requested review from angelathe and vochaparro2 and removed request for a team May 10, 2026 01:37
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented May 10, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 85.29%. Comparing base (2f59e6f) to head (5ba666f).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1459      +/-   ##
==========================================
- Coverage   86.40%   85.29%   -1.11%     
==========================================
  Files         259       24     -235     
  Lines       11584     1190   -10394     
  Branches     1343        0    -1343     
==========================================
- Hits        10009     1015    -8994     
+ Misses       1568      175    -1393     
+ Partials        7        0       -7     
Flag Coverage Δ
ecr-viewer ?
ingestion ?
message-parser ?
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 Sentry.
📢 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.

@alismx alismx marked this pull request as draft May 10, 2026 02:05
@alismx alismx changed the title feat: Add eCR Viewer ECS deployment configuration chore: remove Required column from Variable Groups tables in README May 12, 2026
alismx added 2 commits May 12, 2026 01:06
Rewrite deployment/ecs/README.md from reference-style to workflow-first
structure modeled on deployment/vm/README.md. Adds 5 new sections
(Quick Start, Architecture, Deployment Steps, File Layout,
Troubleshooting), expands Prerequisites to 8-item checkbox checklist,
absorbs Registration into Deployment Steps, renames "Service-Specific
Notes" to "Service-Specific Configuration", and wires cross-references
between sections.

Total: 200 insertions, 48 deletions across 15+ sections.
# Conflicts:
#	deployment/ecs/README.md
@alismx alismx changed the title chore: remove Required column from Variable Groups tables in README refactor: restructure ECS README into user-focused deployment guide May 12, 2026
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