fix(registrar): save the captured beneficiary photo on registration - #91
fix(registrar): save the captured beneficiary photo on registration#91gkbishnoi07 wants to merge 1 commit into
Conversation
|
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 |
bfc6ecf to
f9a9ec9
Compare
…hoto the migration dropped benImage from the registration payload, so the captured photo was never stored and showed as "not found". restore benImage (and changeInBenImage) in the payload and add the missing imageChangeFlag control. also fix the search-result image view to read the photo from the wrapped response (data.data.benImage). fixes MMU-UI QA testing point 5 (registrar side).
f9a9ec9 to
5a86751
Compare
|
1 participant
QA reference
Part of point 5 in the MMU-UI QA testing doc:
This PR is the save side of that fix (the display side is a paired MMU-UI PR).
Root cause
The migration's rewritten
iEMRForm()payload builder dropped the image field. OldmainsentbenImage: personalForm.image(+changeInBenImage), but the migrated payload sent nothing — so the captured photo was never stored, and appeared as "Image not found" everywhere.Changes
registration.component.ts— restorebenImage(+changeInBenImage) in the registration payload, read from theimage/imageChangeFlagcontrols.personal-information.component.ts— add the missingimageChangeFlagcontrol. It was beingpatchValue'd on capture but never existed as a control, so the "image changed" flag silently did nothing (matters for the edit/revisit flow).Verified
After this change the
getBenImageresponse returns the stored base64 photo, and it renders in the nurse worklist view / thumbnail / profile (with the paired MMU-UI display fixes).Paired PR
MMU-UI PR (worklist read path, camera-dialog viewer, profile read path, worklist thumbnail) + the Common-UI pointer bump.