#14143 - Added region/district inputs for quick immunization#14161
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughQuickImmunizationCreationForm's FieldsComponent adds responsibleRegion and responsibleDistrict combo boxes grouped in a jurisdictionRow. These fields are bound with required validation, populated from region/district reference data, dynamically filtered on region change, and shown or hidden based on whether the ImmunizationDto already has jurisdiction values set. ChangesJurisdiction fields cohort
Estimated code review effort: 2 (Simple) | ~15 minutes Sequence Diagram(s)sequenceDiagram
participant User
participant FieldsComponent
participant QuickImmunizationCreationForm
User->>FieldsComponent: open quick immunization form
QuickImmunizationCreationForm->>FieldsComponent: setDto(dto)
QuickImmunizationCreationForm->>QuickImmunizationCreationForm: refreshDistrictOptions()
QuickImmunizationCreationForm->>QuickImmunizationCreationForm: updateJurisdictionFieldsVisibility(dto)
FieldsComponent-->>User: show/hide jurisdictionRow
User->>FieldsComponent: select responsibleRegion
FieldsComponent->>QuickImmunizationCreationForm: region value changed
QuickImmunizationCreationForm->>QuickImmunizationCreationForm: refreshDistrictOptions(region)
FieldsComponent-->>User: updated responsibleDistrict options
Related issues: Suggested labels: ui, bug-fix Suggested reviewers: none identified 🐰 A form once locked with fields unseen, 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Fixes #14143
Summary by CodeRabbit
New Features
Bug Fixes