Skip to content

Add correct phone finder help center link#8086

Merged
kbighorse merged 9 commits intomainfrom
LG-9051-phone-finder-help-center-link
Mar 30, 2023
Merged

Add correct phone finder help center link#8086
kbighorse merged 9 commits intomainfrom
LG-9051-phone-finder-help-center-link

Conversation

@kbighorse
Copy link
Contributor

🎫 Ticket

Link to the relevant ticket.
https://cm-jira.usa.gov/browse/LG-9051

🛠 Summary of changes

This PR replaces an obsolete help center URL with the correct one. It also preserves the internal redirect behavior to the external link and the previous analytics behavior.

changelog: Internal, Identity verification, Fix link to help center articles for identity verification phone finder step
@aduth
Copy link
Contributor

aduth commented Mar 28, 2023

Based on failing Ruby specs, looks like there's a couple other remaining references to the old path.

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.

LGTM 👍

@kbighorse kbighorse merged commit 11e1d44 into main Mar 30, 2023
@kbighorse kbighorse deleted the LG-9051-phone-finder-help-center-link branch March 30, 2023 17:24
),
text: t('idv.troubleshooting.options.learn_more_verify_by_phone'),
new_tab: true,
location: local_assigns.fetch(:name, 'warning'),
Copy link
Contributor

Choose a reason for hiding this comment

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

I missed previously, but this also is in the troubleshooting option, not the URL options, so would not be tracked as expected.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

see #8115

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.

2 participants