entity icon indicating copy to clipboard operation
entity copied to clipboard

UI - Review and Confirm Screen (attention / ref)

Open RFK250 opened this issue 3 years ago • 12 comments

AC

Attn/Ref Component: GIVEN I am reviewing my transfer WHEN I want to enter an attention/ref number THEN I can see attention/ref elements AND I can enter an attention/ref number AND If the length exceeds 40 chars THEN I receive the standard max character field error message/field level error style as I type (user can type more than 40) AND if I delete the extra characters then the field error message goes away

AND

IF there is a max character error in the Attn/Ref field AND I click the Register Changes and Pay button THEN the message "< Please complete required information" appears under the button AND the user is auto-scrolled to the first component with an error AND the component will show a red vertical error line and red subhead as per design comps AND if I complete all page errors then all the error red/messages go away

AC

Confirm Component: IF the user has not checked the Confirm component check box AND I click the Register Changes and Pay button THEN the message "< Please complete required information" appears under the button AND the user is auto-scrolled to the first component with an error AND the component will show red text next to the checkbox, a red vertical error line and red side subhead as per design comps AND if I complete all page errors then all the error red/messages go away

AC

Authorization Component: IF the user has not checked the Authorization component check box AND I click the Register Changes and Pay button THEN the message "< Please complete required information" appears under the button AND the user is auto-scrolled to the first component with an error AND the component will show the text next to the checkbox as red and show a red vertical error line and red side subhead as per design comps AND if I complete all page errors then all the error red/messages go away

RFK250 avatar Oct 12 '22 17:10 RFK250

Hey team! Please add your planning poker estimate with Zenhub @cameron-freshworks @ChasBoucher @chdivyareddy @dimak1 @doug-lovett

RFK250 avatar Oct 12 '22 17:10 RFK250

@dimak1 , When the reference numbers reach the max limit, then the Max error message is displayed which is working as expected. But, 'Please make any required changes' error message is also triggered under the 'Register and Pay changes' button before clicking it. Please take a look, thanks!!

image.png

chdivyareddy avatar Oct 28 '22 18:10 chdivyareddy

@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 message (under the buttons). If it's not the case, maybe we can update the AC @tlebedovich

dimak1 avatar Oct 28 '22 18:10 dimak1

@dimak1 @chdivyareddy - Theses should be separate error triggers. If user errors more than the max character, just show the max character error message under the field (no red side label no red bar or red message under blue button).

If user ignores the max error field message and clicks the Register Changes and Pay button, that is when the red message appears under the button, and the user gets auto-scrolled to the first component with the error, and that's also when the red side bar and red side header also appear.

tlebedovich avatar Oct 28 '22 20:10 tlebedovich

@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? Thanks cc: @chdivyareddy

dimak1 avatar Oct 31 '22 17:10 dimak1

@dimak1 - AC updated for all review and confirm components

tlebedovich avatar Oct 31 '22 17:10 tlebedovich

Increasing Estimate to 3 because of updated AC.

dimak1 avatar Oct 31 '22 18:10 dimak1

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 ?

dimak1 avatar Nov 01 '22 19:11 dimak1

Still being blocked by ongoing Auth/DB issues. cc: @PCC199 @mstanton1

dimak1 avatar Nov 03 '22 22:11 dimak1

Noted: Work is ongoing to fix the Auth/DB issue.

PCC199 avatar Nov 03 '22 22:11 PCC199

Good news - looks like it is working now and this task is unblocked. cc: @PCC199 @mstanton1

dimak1 avatar Nov 04 '22 01:11 dimak1

@dimak1 , When the attention/ref number which is entered in the field and Cancelled the transfer by clicking on the 'Cancel' button, then when the user starts filing another transfer, the ref number from the previous filing is auto-displayed in the field. Please take a look, thanks!!

image.png

  • When the user starts filing another transfer: Same ref number as above image.png

  • When the attention/ref number exceeds 40 chars and the user clicks on the Register and Pay button, then the error message "< Please complete required information" is not appearing under the button. (Instead Please make any required changes message appears and disappears in a few secs). The same error message should display when Confirm Completion component is not checked: image.png

chdivyareddy avatar Nov 07 '22 23:11 chdivyareddy

@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.

dimak1 avatar Nov 08 '22 23:11 dimak1

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 be showing up before we hide the Summary Fee section (until the user makes a change) and update validations for Transfer Details. This way you can verify the error messages on both the initial and Review screens. To get to the Review screen, please make some changes in Home Owners table and then fill out the Transfers Details.

Screen Shot 2022-11-08 at 3.07.32 PM.png

dimak1 avatar Nov 08 '22 23:11 dimak1

Hey @dimak1 , When the 'Attention or Reference Number' is more than 40 then Max char validation is displayed, but when the user clicks on the 'Register changes and Pay' button, error message under fee summary is populated for a sec and disappears without auto-scrolling to the component. Please take a look, thanks!

image.png

When Register changes and pay button is clicked: No error message and not auto-scrolled to the 'Attention or Reference Number' component image.png

chdivyareddy avatar Nov 11 '22 20:11 chdivyareddy

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 check, thanks!

dimak1 avatar Nov 11 '22 20:11 dimak1

@chdivyareddy I fixed the validation for the Attention Reference Number field. However, submission in my local dev env is not going thru. It could be because it is local env or maybe there are some other issues that are beyond this ticket. Once we merge PRs 1036 and 1037 and submit in Test env, we would be in a better position to tell.

dimak1 avatar Nov 14 '22 05:11 dimak1

Verified!

When the user clicks the 'Register changes and pay' button without completing the required component, then the user is auto-scrolled to the first component with an error and will show a red vertical error line:

  • Attention/ref number: image.png

  • Confirm Component: image.png

  • Authorization Component: image.png

chdivyareddy avatar Nov 14 '22 23:11 chdivyareddy