-
Notifications
You must be signed in to change notification settings - Fork 298
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
Ako/ CRO-724/ Pass growthbook attributes on initialisation #16713
Ako/ CRO-724/ Pass growthbook attributes on initialisation #16713
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
A production App ID was automatically generated for this PR. (log)
Click here to copy & paste above information.
|
🚨 Lighthouse report for the changes in this PR:
Lighthouse ran with https://deriv-app-git-fork-ali-hosseini-deriv-ako-cro-724use-ana-aaa8a1.binary.sx/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Neat!
@@ -1,5 +1,5 @@ | |||
import { Analytics } from '@deriv-com/analytics'; | |||
import Cookies from 'js-cookie'; | |||
import * as Cookies from 'js-cookie'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[Q] do we use all?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actually Not needed. let me remove it
650ee96
Quality Gate passedIssues Measures |
* build: update analytics to ver 1.13.0 * build: update to version 1.14 * feat: add user_id only if its available * test: fix analytics test cases * feat: pass the attributes on GB initialisation * build: match the js-cookie types with the package version * fix: remove country property as its built in to the analytics package * build: update to 1.15.0 to also include changes for the new location implementation * chore: revert the icons file changes * Revert "build: update to 1.15.0 to also include changes for the new location implementation" This reverts commit 67445cb. * Revert "fix: remove country property as its built in to the analytics package" This reverts commit e8dcc9e. * chore: change import statement
Changes:
This is to ensure we initialise the rudderstack and growthbook at the same time. which will fix the Growthbook Experiments issue.
We also dont need to pass the country location from initialise function anymore as it will picked up automatically on analytics package.