Skip to content

Add properties to data retention banner custom event#9063

Merged
mehansen merged 2 commits into
mainfrom
merethe/data-retention-event-properties
Aug 8, 2025
Merged

Add properties to data retention banner custom event#9063
mehansen merged 2 commits into
mainfrom
merethe/data-retention-event-properties

Conversation

@mehansen
Copy link
Copy Markdown
Contributor

@mehansen mehansen commented Aug 8, 2025

FRONTEND PULL REQUEST

Related Issue

request from Lauryn to improve analytics of user response to data retention communication

Changes Proposed

add properties to the custom event tracked when users click through to the support site from the data retention limits banner to differentiate between clicks from the patients page and the results page

Testing

deployed to a lower and see if app insights shows the correct properties for the event(s)

Screenshots / Demos

Screenshot 2025-08-08 at 12 01 31 PM

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Aug 8, 2025

Copy link
Copy Markdown
Collaborator

@DavidMcClatchey DavidMcClatchey left a comment

Choose a reason for hiding this comment

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

lgtm 🚀

Copy link
Copy Markdown
Collaborator

@kevinfiol kevinfiol left a comment

Choose a reason for hiding this comment

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

Looks good.

Copy link
Copy Markdown
Contributor

@arinkulshi-skylight arinkulshi-skylight left a comment

Choose a reason for hiding this comment

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

LG! working in dev3

@mehansen mehansen added this pull request to the merge queue Aug 8, 2025
Merged via the queue into main with commit d8d69ff Aug 8, 2025
45 checks passed
@mehansen mehansen deleted the merethe/data-retention-event-properties branch August 8, 2025 19:29
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.

4 participants