Skip to content

GA4 docs updates #2787

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 12 commits into from
May 2, 2022
Merged

GA4 docs updates #2787

merged 12 commits into from
May 2, 2022

Conversation

kdaswani
Copy link
Contributor

@kdaswani kdaswani commented Apr 22, 2022 •
edited
Loading

Proposed changes

More updates to GA4 as our integration and Google's API advances. This is ready for review and can be deployed in the next deploy on May 3!

Merge timing

  • Tuesday, May 3

Changes to make:

  • Event Parameters do not support nested objects - event will be silently dropped
  • How to send session_id
  • Engagement time parameter to count users
  • Sending user properties

@kdaswani kdaswani requested a review from a team as a code owner April 22, 2022 22:04
@kdaswani kdaswani requested review from rchinn1 and removed request for a team April 22, 2022 22:04
kdaswani
kdaswani commented Apr 29, 2022
View reviewed changes
@kdaswani kdaswani changed the title Draft of GA4 changes GA4 docs updates Apr 29, 2022
@kdaswani
Copy link
Contributor Author

@markzegarelli @stayseesong this is ready for review! Thank you!

rchinn1
rchinn1 reviewed May 2, 2022
View reviewed changes
Copy link
Contributor

@rchinn1 rchinn1 left a comment

Choose a reason for hiding this comment

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

Looks good to me! Just left a few suggested changes.

@kdaswani
Copy link
Contributor Author

kdaswani commented May 2, 2022

Thank you @rchinn-segment! Applied the changes!

stayseesong
stayseesong requested changes May 2, 2022
View reviewed changes
src/connections/destinations/catalog/actions-google-analytics-4/index.md Outdated
@@ -10,10 +10,10 @@ id: 60ad61f9ff47a16b8fb7b5d9
When you have Segment installed, you can use your existing tracking implementation to fulfill your data collection needs with Google Analytics 4. Segment will send your data server-side to [Google's Measurement Protocol API](https://developers.google.com/analytics/devguides/collection/protocol/ga4).

> warning ""
> You won't be able to see user data or realtime data in your Google Analytics 4 instance, as Google's API is [in beta](https://developers.google.com/analytics/devguides/collection/protocol/ga4/reference/limitations){:target="_blank"} and hasn't added session support. You can contact Google for more information.
> Google Analytics 4 does not officially support a pure server-to-server integration, however Segment continues to monitor the capabilities of the Measurement Protocol API and will make updates accordingly to achieve a reasonable level of reporting for our mutual customers. Segment does not have plans to build a device-mode integration with Gtag for Google Analytics 4.

This comment was marked as outdated.

Sign in to view

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Already accepted a different update to this that I think works!

@kdaswani kdaswani requested a review from stayseesong May 2, 2022 20:31
@kdaswani
Copy link
Contributor Author

kdaswani commented May 2, 2022 •
edited
Loading

Thanks for approving @stayseesong! Can we merge this in so that it can be part of tomorrow's deploy? Appreciate the thorough review from the team!

@kdaswani kdaswani merged commit ca247ae into develop May 2, 2022
@kdaswani kdaswani deleted the ga4-updates branch May 2, 2022 23:20
@github-actions
Copy link
Contributor

github-actions bot commented May 2, 2022

Thank you for your contribution! Your pull request is merged, but may take a day or two to appear on the site.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@markzegarelli markzegarelli markzegarelli approved these changes

@stayseesong stayseesong stayseesong approved these changes

@rchinn1 rchinn1 rchinn1 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@kdaswani @markzegarelli @stayseesong @rchinn1