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
Add docUploadOwner and docUploadReviewOwner screens #704
Conversation
spokenbird
commented
Apr 16, 2024
- Adds the above screens with the goal of associating a household member with an uploaded document.
- Create associations between household member and uploaded document
condition: ContinueAssociateDocType | ||
- name: docUpload | ||
docUploadReview: | ||
docUploadOwner: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tiny, non blocking. This reads easier when the sections are in order. I'd expect docUploadReviewTypes to be in line 542 and then docUploadOwner to be listed afterward.
helpText='', | ||
content=~{::docWho}, | ||
ariaDescribedBy=${fileId}, | ||
label='', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we can typically omit label when ariaLabel exists.
LGTM. I would love to test once the tests pass and we can deploy |
QA'ed and looks good! |