Skip to content
This repository has been archived by the owner on Sep 26, 2024. It is now read-only.

Amir/CRO-300/Add page tracking event #6013

Merged
merged 1 commit into from
Nov 24, 2023
Merged

Amir/CRO-300/Add page tracking event #6013

merged 1 commit into from
Nov 24, 2023

Conversation

amir-deriv
Copy link
Contributor

Changes:

  • Add page tracking event on deriv.com
  • Previously on update of every route, we were sending this analytical data to GTM, now to track the whole user journey we are also using rudderstack events to send this data to different sources

Following is the screenshot from rudderstack, we can see the events are getting there on our rudderstack app

image

Type of change

  • Bug fix
  • New feature
  • Update feature
  • Refactor code
  • Translation to code
  • Translation to crowdin
  • Script configuration
  • Improve performance
  • Style only
  • Dependency update
  • Documentation update
  • Release

@vercel Vercel
Copy link

vercel bot commented Nov 23, 2023 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
deriv-com ✅ Ready (Inspect) Visit Preview Nov 23, 2023 3:14am

@sonarqubecloud SonarQubeCloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
5.9% 5.9% Duplication

@github-actions GitHub Actions
Copy link
Contributor

Preview Link: https://amir-cro-300-impleme.deriv-com-preview-links.pages.dev

Name Result
Build status Completed ✅
Preview URL Visit Preview
Action URL Visit Action

@github-actions GitHub Actions
Copy link
Contributor

Running Lighthouse audit...

@github-actions GitHub Actions
Copy link
Contributor

🚀 Smoke test run (4) passed successfully!

@meenakshi-deriv meenakshi-deriv merged commit f530998 into binary-com:master Nov 24, 2023
6 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@markw-deriv markw-deriv markw-deriv approved these changes

@fasihali-deriv fasihali-deriv fasihali-deriv approved these changes

@niloofar-deriv niloofar-deriv niloofar-deriv approved these changes

@habib-deriv habib-deriv Awaiting requested review from habib-deriv habib-deriv is a code owner

@prince-deriv prince-deriv Awaiting requested review from prince-deriv prince-deriv is a code owner

@mohsen-deriv mohsen-deriv Awaiting requested review from mohsen-deriv

@mitra-deriv mitra-deriv Awaiting requested review from mitra-deriv mitra-deriv is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@amir-deriv @niloofar-deriv @fasihali-deriv @markw-deriv @meenakshi-deriv