COI: Building front-end (Employee Disclosure Statement)

[ayushdamani]

Build the Employee Disclosure Statement tab

Acceptance Criteria

• [x] The Employee Disclosure Statement tab is built as per the design and content provided below
• [x] Questions 1, 2 and 3 should be text area fields and mandatory. (no technical limit on word count for DJ)
• [x] Question 4 should be a file attachment field and optional (file size and types TBD)
• [x] Question 5 should be radio button (Yes/no) and mandatory
• [x] If Question 5 answer is Yes, only then Question 6 should be shown
• [ ] If Question 5 answer is Yes only then Question 6 is mandatory i.e there must be an attachment (file size and types TBD)

Design

Link to miro - https://miro.com/app/board/uXjVKbE3n_8=/?moveToWidget=3458764585134827511&cot=14

[jerry-ey]

Please test

[fazil-ey]

1. Question 5 should be mandatory and Question 6 if it shows should be mandatory https://bcgov.sharepoint.com/:v:/r/teams/03991/Shared%20Documents/Conflict%20of%20Interest/Test%20recordings%20and%20screenshots/Screen%20Recording%202024-04-22%20at%208.03.56%20PM.mov?csf=1&web=1&e=zV98RM

2. Seeing this unauthorized message when uploading files

   

3. Can we remove the file name and size box

   

4. Change the supported types message to

Supported file types: .pdf, .png, .jpg/jpeg, .docx, .eml, .msg

4. Change the error message to

Unsupported file type. File must be .pdf, .png, .jpg/jpeg, .docx, .eml, .msg

[fazil-ey]

file upload needs to be handled in #1720

[MikePSA]

1. "Please provide the following" is missing from the top of the Employee Disclosure Statement tab, according to the design on COI miro. (Ignore if doesn't matter)

   

2. Unauthorized message still shows for both questions 4 & 6 when uploading files

   

3. If you try to proceed without answering question 5, the validation error repeats the same question. Should be changed.

   

Use - "This field is required"

4. If you try to proceed without answering question 6, the validation error repeats supported file types. However, in this example I have not attached anything. Validation error message is not necessarily correct in this scenario.

Use "Third Party Authorization form is required"

[jerry-ey]

@MikePSA , For 2. and all attachment related, please refer to other ticket as mentioned above in the comment.

[jerry-ey]

please test again.

[jerry-ey]

Note to me: To fix the number 4 item. Check the mat/par parental leave for example. can do error message based on different condition.

[jerry-ey]

@fazil-ey Based on source code of Formio, we are not allowed to change the following error message for file upload component.

• file type
• file size

But only can change the "required" error message.

[fazil-ey]

The error message is correct now when no file is attached