Skip to content

LG-4826: Remove reference to hello@login.gov#5598

Merged
mdiarra3 merged 7 commits intomainfrom
LG-4826-remove-login-email
Nov 15, 2021
Merged

LG-4826: Remove reference to hello@login.gov#5598
mdiarra3 merged 7 commits intomainfrom
LG-4826-remove-login-email

Conversation

@mdiarra3
Copy link
Contributor

@mdiarra3 mdiarra3 commented Nov 9, 2021

Why?

Trying to remove all reference to Hello@login.gov in IDP.

@mdiarra3 mdiarra3 requested review from aduth and jmdembe November 9, 2021 18:38
Copy link
Contributor

@aduth aduth left a comment

Choose a reason for hiding this comment

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

👍

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

Comment on lines +115 to +119
attestation_error: Sorry, but your security key doesn’t appear to be a FIDO
security key. Please make sure your device is listed at
https://fidoalliance.org/certification/fido-certified-products/ and if
you believe the error is ours, please contact us at hello@login.gov.
you believe the error is ours, please contact at
https://login.gov/contact/
Copy link
Contributor

Choose a reason for hiding this comment

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

Do you think it would make sense to make these URLs interpolated? ex %{fido_url} and %{contact_url} and then we can use like MarketingSite class that auto inserts the locale?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yup we already ahve the contact url so i updated and pushed up that change .

@mdiarra3 mdiarra3 merged commit dfec5fe into main Nov 15, 2021
@mdiarra3 mdiarra3 deleted the LG-4826-remove-login-email branch November 15, 2021 14:20
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.

5 participants