Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Supporting Info section of review and confirm displays "No docs required" and list of docs #24940

Open
forgeuxGH5 opened this issue Dec 13, 2024 · 2 comments
Labels
bug Something isn't working hosts STR2 Short Term Rental Registry workstream #2

Comments

@forgeuxGH5
Copy link
Collaborator

Step to reproduce

  • Enter an address without exemption that requires documents to be uploaded
  • Upload required documents
  • Select Property is Exempt (Step 1)
  • Select Farm land
  • View Supporting Documentation (Step 3) = No documentation required with no documents listed (and no way to remove previously uploaded documents)

image.png

  • View Review and Confirm (Step 4) - Supporting Information = see both No documentation required AND previously upload documents

image.png

Expected behaviour
Review and Confirm (Step 4) - Supporting Information = displays ONLY "No documentation required" (preferred)
OR
Step 3 displays No documentation required but allows removal of previously added documents

@forgeuxGH5 forgeuxGH5 added bug Something isn't working hosts STR2 Short Term Rental Registry workstream #2 labels Dec 13, 2024
@forgeuxGH5 forgeuxGH5 changed the title Supporting Doc section of review and confirm retains values after changing exemption reason Supporting Info section of review and confirm displays "No docs required" and list of docs Dec 13, 2024
@deetz99
Copy link
Collaborator

deetz99 commented Dec 13, 2024

@forgeuxGH5

In this scenario you described, should we delete any documents that were previously uploaded? Or just have the 'No documentation required' label? (but the documents would still be uploaded with their application)

@forgeuxGH5
Copy link
Collaborator Author

delete - those docs would go to the Examiners and confuse them

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working hosts STR2 Short Term Rental Registry workstream #2
Projects
None yet
Development

No branches or pull requests

2 participants