-
Notifications
You must be signed in to change notification settings - Fork 64
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
chore: Add Hotjar integration #654
chore: Add Hotjar integration #654
Conversation
@2dpetkov is attempting to deploy a commit to the dYdX Trading Team on Vercel. A member of the Team first needs to authorize it. |
da75a2a
to
857ee56
Compare
857ee56
to
61fe70e
Compare
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
@@ -37,6 +37,7 @@ jobs: | |||
run: | | |||
pnpm run build | |||
pnpm run build:inject-amplitude | |||
pnpm run build:inject-hotjar |
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.
fyi that this workflow is no longer used!
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.
shall I then drop this change?
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.
all good to keep -- i'll do another pass later on to remove unused workflows!
|
||
<b>To use with dydxprotocol/v4-web:</b> | ||
1. Hotjar account with Site Id and Hotjar version. | ||
2. Add Site Id and Hotjar version in Github > Secrets and Variables > Actions as `HOTJAR_SITE_ID` and `HOTJAR_VERSION` |
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.
ah the instruction for amplitude was outdated - we can keep this, but in reality the variables only need to be added on vercel
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.
confirming that DOS's prod deployment will have that in Vercel ✅
fix typo Co-authored-by: aleka <aforaleka@gmail.com>
build:inject-analytics
npm script, so that the Vercel custom build command limitation of 256 characters is not hit when all scripts are included