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
Implement basic doc upload page and tests #242
Conversation
nextScreens: | ||
- name: doc-upload-confirm | ||
doc-upload-confirm: | ||
condition: SkipTemplate |
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.
removed skip template because otherwise there is no nextScreen for doc-upload-add-files
doc-upload-add-files.header=Upload documents | ||
doc-upload-add-files.accordion-1.title=Recommended documents | ||
doc-upload-add-files.accordion-1.p1=Share documents that verify: | ||
doc-upload-add-files.accordion-1.body1= <strong>Parent jobs and income</strong><br/>Add two recent pay stubs or an income verification letter for each working parent. |
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.
Added each paragraph in its separate string just in case there are copy changes: It will affect a smaller grouping of words rather than all the text.
@@ -105,12 +103,30 @@ protected void uploadFile(String filepath, String dzName) { | |||
.isBlank()); | |||
} | |||
|
|||
protected void uploadFile(String filepath) { |
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.
There are similar methods with different signatures that may be tied to drag and drop.
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.
Jira issue is missing:
-
doc-upload-recommended-docs
screen - corresponding Journey test
<p th:utext="#{doc-upload-add-files.accordion-1.body5}"></p> | ||
</th:block> | ||
</th:block> | ||
<th:block th:replace="~{fragments/fileUploader :: fileUploader(inputName='uploadDocuments')}"></th:block> |
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.
The file uploader implementation should be implemented in the next ticket. Not a problem now, in my opinion
ac78a1b
to
e0b0a41
Compare
f5f6ff6
to
3bfe2b1
Compare
69a8a24
to
84c8e36
Compare
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.
The copy forTANF Work Training
is incorrect on the doc-upload-recommended-docs
screen. Everything else is good to go
<p th:utext="#{doc-upload-recommended-docs.school.body}"></p> | ||
|
||
<th:block th:replace="~{fragments/gcc-icons :: job}"></th:block> | ||
<p th:utext="#{doc-upload-recommended-docs.school.body}"></p> |
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.
I think you meant to doc-upload-recommended-docs.training.body
instead of doc-upload-recommended-docs.school.body
.
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.
Thank you! @enyia21
🔗 Jira ticket
Issue Link
✍️ Description
📷 Design reference
Figma Design
✅ Completion tasks
Note: If you are testing this locally, you will need to copy over staging AWS credentials.