department-of-veterans-affairs / va.gov-cms

Editor-centered management for Veteran-centered content.
https://prod.cms.va.gov
GNU General Public License v2.0
97 stars 69 forks source link

[Analytics] [Translation Widget] Custom GA events removal & va-link component w/analytics adoption #18207

Open FranECross opened 4 months ago

FranECross commented 4 months ago

Status

[2024-07-18] [Fran] Fran to check the spreadsheet to ensure this has been added, and then will close. [2024-07-18] [Fran] Per Chris K, there's a comment from Laura, that we cannot implement this yet due to va-link not able to take in the lang attribute through to the . Can you close this once the spreadsheet is updated?

Description

The va-link component has been enhanced to provide destination and can be used now for the Translation Widget, as well as the existing custom events removed.

User story

AS A PO/PM managing Public Websites products & features I WANT to (where appropriate) adopt the va-link component, its baked-in analytics, and remove the custom GA events SO THAT extraneous code is removed and only the events needed will be triggered.

Engineering notes / background

Analytics considerations

Quality / testing notes

Acceptance criteria

chriskim2311 commented 3 months ago

@FranECross I have a comment from Laura here, that we cannot implement this yet due to va-link not able to take in the lang attribute through to the <a>. Can you close this once the spreadsheet is updated.