-
Notifications
You must be signed in to change notification settings - Fork 128
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
[99786] Appoint a Rep add submission types to search results #34105
[99786] Appoint a Rep add submission types to search results #34105
Conversation
const submissionTypeContent = v2IsEnabled && | ||
userIsDigitalSubmitEligible && ( | ||
<p data-testid="submission-methods"> | ||
Accepts VA Form {formNumber} online, by mail, and in person |
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.
assume for now that all reps/orgs can accept all types
There will be some cleanup/refactoring to do once the submission method storage is implemented |
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.
Approved
dc62e55
to
4f92631
Compare
4f92631
to
566465b
Compare
Requesting re-review after resolving merge conflicts |
* add submission types to search results * fix spec nesting
Are you removing, renaming or moving a folder in this PR?
Did you change site-wide styles, platform utilities or other infrastructure?
Summary
Related issue(s)
Testing done
Screenshots
Note: This field is mandatory for UI changes (non-component work should NOT have screenshots).
What areas of the site does it impact?
Appoint a Rep Form 21-22 and 21-22a
Acceptance criteria
Quality Assurance & Testing
Error Handling
Authentication