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 global key #37

Merged
merged 5 commits into from
Feb 4, 2025
Merged

Add analytics global key #37

merged 5 commits into from
Feb 4, 2025

Conversation

@nunofgs nunofgs force-pushed the add-analytics-global-key branch from 28910d0 to 4a11107 Compare January 30, 2025 12:18
.mise.toml Outdated
@@ -0,0 +1,5 @@
[env]
NODE_OPTIONS = "--openssl-legacy-provider"
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

This is unrelated to this PR but needed to build the browser dist by running npm run build. I can remove it from this PR if the reviewer prefers.

@@ -0,0 +1,5 @@
---
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

There are lots of seemingly unrelated changes. This PR is simply the result of:

$ git cherry-pick 7f4232cbdb60a4475c565e5d262b25182e47baf4
$ git cherry-pick 5c1511fe1e1d1df94967623b29ec12ffe770aacf

And the fixing merge conflicts.

@nunofgs nunofgs changed the base branch from main to allow-localhost-urls-for-analytics-js January 30, 2025 13:15
@nunofgs nunofgs force-pushed the add-analytics-global-key branch from 4a11107 to e4d7cdc Compare January 30, 2025 13:16
Base automatically changed from allow-localhost-urls-for-analytics-js to main January 31, 2025 15:03
Copy link

@jatinn jatinn left a comment

Choose a reason for hiding this comment

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

lgtm not best at js, but if i understood whats happening is basically allowing using any name for the analytics and introduces the getGlobalAnalytics func that replaces the window.analytics in the rest of the code

@nunofgs nunofgs merged commit 1a3c81e into main Feb 4, 2025
1 check passed
@nunofgs nunofgs deleted the add-analytics-global-key branch February 4, 2025 17:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants