Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Analytics call to Track Deletion of Templates #3971

Merged
merged 5 commits into from
Aug 16, 2023
Merged

Add Analytics call to Track Deletion of Templates #3971

merged 5 commits into from
Aug 16, 2023

Conversation

Cliftonz
Copy link
Contributor

@Cliftonz Cliftonz commented Aug 14, 2023

What change does this PR introduce?

This PR introduces the addition of analytics tracking for the deletion of templates.

Why was this change needed?

Allows us to track the analytics on deleting notfication templates so that:

  1. User Behavior Insight: By tracking the deletion of templates, we gain deeper insights into user behavior and understand which templates are being frequently deleted and why.

  2. Enhancing User Experience: By understanding the patterns around template deletions, we can provide more tailored recommendations or warnings to users, ensuring a smoother and more intuitive user experience.

@Cliftonz Cliftonz changed the title Add Analytics call to track deletion of templates Add Analytics call to Track Deletion of Templates Aug 14, 2023
data: {
createdAt: item.createdAt,
lastUpdatedAt: item.updatedAt,
deletedAt: new Date(),
Copy link
Contributor

Choose a reason for hiding this comment

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

No need to pass this, as the time is already created by segment and downstream providers

@Cliftonz Cliftonz marked this pull request as ready for review August 16, 2023 21:21
@Cliftonz Cliftonz added this pull request to the merge queue Aug 16, 2023
Merged via the queue into next with commit aea01d7 Aug 16, 2023
@Cliftonz Cliftonz deleted the analytics branch August 16, 2023 21:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants