Dima K
Dima K
@chdivyareddy as per the last item in the AC, it should prevent us from the submission in case there are errors. My understanding was that we should show that error...
@tlebedovich thanks for the clarification. Based on this, I should probably refactor Confirm Completion section validation as well. Could you please update AC for both while I start updating validation?...
Increasing Estimate to 3 because of updated AC.
Added all validation errors and scroll to the first error. However, there is a dependency on the Transfer Details validation #14091. Should we pull this in as well @mstanton1 ?
Still being blocked by ongoing Auth/DB issues. cc: @PCC199 @mstanton1
Good news - looks like it is working now and this task is unblocked. cc: @PCC199 @mstanton1
@chdivyareddy update both issues. Fixed Attn Num from reappearing for different Transfers and updated the error message on the Review screen. PR is under review.
Also, @chdivyareddy heads up that you might see the error coming up under Summary Fee (as in the screenshot below). It is normal at the moment, and it will not...
Hi @chdivyareddy, yes I'm able to reproduce it myself. I can see that it's trying to submit the Transfer ignoring the error in Attn Ref field. Validation issues. I will...
@chdivyareddy I could comment on that. 1. For Hide/Show Transfers Details there was a dependency on another ticket (which was recently completed) so I could go back and finish this...