-
Notifications
You must be signed in to change notification settings - Fork 1
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 new ga4 events #161
Add new ga4 events #161
Conversation
…matt-dev-add-new-ga4-events
…matt-dev-add-new-ga4-events
…matt-dev-add-new-ga4-events
…matt-dev-add-new-ga4-events
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.
Great work man. I know this was a lot of effort and it's much appreciated!
To Do:
1: There are a few minor requests to address in the comments.
2: There is a merge conflict in a single file that needs to be resolved.
Note: One thing to be minful of, which I did not see, but maybe you want ot make a quick pass over, is to ensure that only things were added. In that, no logic paths were changed as a side effect.
…matt-dev-add-new-ga4-events
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.
Addressed typo, Removed extra braces, Resolved merge conflict (due to recent PR merge), Approved, Merging now
No description provided.