fix(anc): require Primi Gravida and LMP in the nurse ANC form - #470
fix(anc): require Primi Gravida and LMP in the nurse ANC form#470gkbishnoi07 wants to merge 1 commit into
Conversation
the nurse ANC obstetric form never marked primiGravida/lmpDate as required, so an empty submit skipped frontend validation and hit the backend with a generic "unable to save data". add the validators the submit check already expects, so the missing-fields dialog shows instead.
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Advanced Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
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 |
|
1 participant
What
Marks Primi Gravida and Last Menstrual Period (LMP) as required in the nurse ANC obstetric form (
createPatientANCDetailsFormingeneral-utility.ts).Why
These two fields drive the ANC obstetric details. When they were left empty, the form-level "required fields" validation never flagged them, so the submit went straight to the backend and returned a confusing generic error. Adding the validators the submit check already looks for means the user now sees a clear "Below Fields Are Required: Primi Gravida, Last Menstrual Period" dialog instead.
Scope
Testing