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

Discovery: FE Alerts and analytic event triggers when PDF downloads fail #10061

Closed wesrowe closed 2 years ago

wesrowe commented 2 years ago

Description

In #10042, Daniel's investigation into GA events cast doubt on the validity of our GA (Google Analytics) events.

Background on GA: The GA event Wes is using to identify PDF download issues tracks an Alert in the ui. GA event: Navigation - Alert Box - Link Click - error - This form link isn’t working - (not set) - email the forms managers. Over time, spikes in this signal correspond anecdotally to VBA Forms Managers receiving emails from Veterans that a PDF download failed. We suspect that the Alert provides a way to email Forms managers, but need to verify.

GA provides the Form Detail pages where this event fired, and about 1/3 of them were from the subdomain www.va.gov, which he verified is an S3 bucket where such failures are extremely improbable.

Questions to investigate on the FE:

On Forms Detail pages:

On the Find-a-form page (ie, list view), which also offers PDF Download links:

API Calls to investigate:

Monitoring setup: Are key interactions being logged to sentry on failure? on success?

Acceptance Criteria

CMS Team

Please check the team(s) that will do this work.

dsasser commented 2 years ago

Follow up tickets created for this discovery work:

10235

10236

10237

10239

dsasser commented 2 years ago

Mural link with discovery findings: https://app.mural.co/t/vagov6717/m/vagov6717/1660150431569/27c80eaeeaedfd88c80322ae177b68c503345ba9?sender=u2cc9d7c4c6c0dea143b21411

Image for those unable to access Mural: VACMS-10061 - Discovery Form FE Workflow

jilladams commented 2 years ago

@dsasser 2 follow up items on this:

  1. The modal has a fallback mode that points veterans to email Forms managers when it fails. Can you add that?
  2. The modal shows once per N (we think 24h) - can you add that too, or a guess about it?