Skip to content

LG-5373: Update troubleshooting options on the IAL2 Add your ID step#5654

Merged
aduth merged 8 commits intomainfrom
aduth-lg-5373-doc-auth-troubleshooting
Dec 3, 2021
Merged

LG-5373: Update troubleshooting options on the IAL2 Add your ID step#5654
aduth merged 8 commits intomainfrom
aduth-lg-5373-doc-auth-troubleshooting

Conversation

@aduth
Copy link
Contributor

@aduth aduth commented Dec 1, 2021

Why: As a user, I want to see options for which ID I can use to verify my identity, and tips for taking photos of my ID, so that I can get more help if I am having trouble adding photos of my ID without abandoning this step to go back to the service provider.

Screenshots:

Step Mobile Desktop
Document Capture localhost_3000_verify_doc_auth_document_capture(iPhone X) localhost_3000_verify_doc_auth_document_capture
Review Issues localhost_3000_verify_doc_auth_document_capture(iPhone X) (1) localhost_3000_verify_doc_auth_document_capture (1)

aduth added 2 commits December 1, 2021 09:40
**Why**: As a user, I want to see options for which ID I can use to verify my identity, and tips for taking photos of my ID, so that I can get more help if I am having trouble adding photos of my ID without abandoning this step to go back to the service provider.
aduth added 5 commits December 1, 2021 11:02
**Why**: Mostly for appeasement of tests, but since URLs default to empty strings, don't bother rendering, since (a) they don't lead anywhere useful and (b) they cause a duplicate key warning in the TroubleshootingOptions component.
Copy link
Contributor

@zachmargolis zachmargolis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

* @typedef MarketingSiteContext
*
* @prop {string} documentCaptureTipsURL Link to Help Center article with tips for document capture.
* @prop {string} supportedDocumentsURL Link to Help Center article detailing supported documents.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

would it make sense to do something like @see AppRootData or something to minimize duplication?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

would it make sense to do something like @see AppRootData or something to minimize duplication?

Hm, yeah, I see the concern about the duplication. I think it might be nice if we could set up AppRootData to be an intersection type of all the different context values it's picking from, or vice-versa this context type a Pick<AppRootData, 'foo'|'bar'|'baz'>. One issue is that the dataset capitalization for the properties is different from the context capitalization, so the names don't line up exactly.

I think it could just be enough to do like you suggest with a @see, even if it's not "linked up" in a way that TypeScript/IDE/etc could follow, and avoid the extended property descriptions in all but the one canonical source.

Comment on lines +64 to +67
* @see AppContext
* @see MarketingSiteContext
* @see FailedCaptureAttemptsContext
* @see UploadContext
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

Copy link
Contributor

@anniehirshman-gsa anniehirshman-gsa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Screenshots LGTM!

Does the "More tips for adding photos of your ID" link lead to https://login.gov/help/verify-your-identity/how-to-add-images-of-your-state-issued-id/ ? Couldn't find it in the code but I may just be missing it

@aduth
Copy link
Contributor Author

aduth commented Dec 2, 2021

Does the "More tips for adding photos of your ID" link lead to https://login.gov/help/verify-your-identity/how-to-add-images-of-your-state-issued-id/ ? Couldn't find it in the code but I may just be missing it

Yep, it does 👍 This had existed already, so wasn't part of the changes here. But the relevant code is here:

def self.help_document_capture_tips_url
URI.join(
BASE_URL,
locale_segment,
'help/verify-your-identity/how-to-add-images-of-your-state-issued-id/',
).to_s
end

@aduth aduth merged commit 95eab34 into main Dec 3, 2021
@aduth aduth deleted the aduth-lg-5373-doc-auth-troubleshooting branch December 3, 2021 13:20
).to_s
end

def self.help_document_capture_supported_documents_url
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I didn't notice an equivalent method was already available above:

def self.help_idv_supported_documents_url
URI.join(
BASE_URL,
locale_segment,
'help/verify-your-identity/accepted-state-issued-identification/',
).to_s
end

I'll take care of this in #5662.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants