-
Notifications
You must be signed in to change notification settings - Fork 367
Update Hotjar destination docs #3137
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
Conversation
Reflect latest changes (events support), remove mentions of tools no longer available in Hotjar, among other updates.
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 looks good, thanks for the quick review 👍
Co-authored-by: stayseesong <[email protected]>
Co-authored-by: stayseesong <[email protected]>
Co-authored-by: stayseesong <[email protected]>
Co-authored-by: stayseesong <[email protected]>
Co-authored-by: stayseesong <[email protected]>
Co-authored-by: stayseesong <[email protected]>
Co-authored-by: stayseesong <[email protected]>
Co-authored-by: stayseesong <[email protected]>
Co-authored-by: stayseesong <[email protected]>
UAs don't need to be enabled manually because Hotjar does it automatically. Also removing the date as suggested.
Co-authored-by: stayseesong <[email protected]>
Co-authored-by: stayseesong <[email protected]>
Thanks @Maguna! We'll roll these changes out with our next deploy on July 5. |
Thank you for your contribution! Your pull request is merged, but may take a day or two to appear on the site. |
@markzegarelli hey quick question could we remove this as well from the page - and ideally from the in-app wizard as well? We don't support it anymore. |
Reflect latest changes (events support), remove mentions of tools no longer available in Hotjar, among other updates.
Proposed changes
Merge timing
Related issues (optional)