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

Use Google Analytics 4 (GA4) ID #25

Merged
merged 1 commit into from
Nov 16, 2023
Merged

Use Google Analytics 4 (GA4) ID #25

merged 1 commit into from
Nov 16, 2023

Conversation

chalin
Copy link

@chalin chalin commented Feb 24, 2023

Signed-off-by: Patrice Chalin <[email protected]>
@netlify
Copy link

netlify bot commented Feb 24, 2023

Deploy Preview for stoic-goldberg-e3a26b ready!

Name Link
🔨 Latest commit f7b05ef
🔍 Latest deploy log https://app.netlify.com/sites/stoic-goldberg-e3a26b/deploys/63f89e7b4f868e0008123996
😎 Deploy Preview https://deploy-preview-25--stoic-goldberg-e3a26b.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@erikbledsoe
Copy link
Collaborator

This was achieved through the GA UA to GA4 migration process. So this could probably be closed.

@chalin chalin changed the title Use Google Analytics 4 (GA4) IDD Use Google Analytics 4 (GA4) ID Nov 15, 2023
@chalin
Copy link
Author

chalin commented Nov 15, 2023

This was achieved through the GA UA to GA4 migration process. So this could probably be closed.

Hi @erikbledsoe - no, this was part of the migration process, but it didn't get merged, and so the migration isn't complete.

The website has active GA4 tags in use (but not the CNCF tag), so either someone set them up or they were the ones that Google automatically created.

/cc @caniszczyk

@erikbledsoe
Copy link
Collaborator

@chalin Ah, my bad. Didn't realize that was the CNCF tag.
@edsiper can you merge this?

@edsiper edsiper merged commit 6814e57 into fluent:master Nov 16, 2023
@edsiper
Copy link
Member

edsiper commented Nov 16, 2023

merged, thanks!

@chalin
Copy link
Author

chalin commented Nov 17, 2023

Thanks @edsiper and @erikbledsoe. As mentioned in

... the docs also need to be updated to use G-93T2VC0VBC. The site is "powered by GitBook". Can one of you change the GA ID being used by that site?

@chalin
Copy link
Author

chalin commented Nov 17, 2023

@edsiper @erikbledsoe - the same is true of the pending task (relative to the GitBook docs subdomain) mentioned in:

Can you here there as well?

@erikbledsoe
Copy link
Collaborator

@chalin I just tried adding a second Google tag into the GitBook GA plugin (on the Fluent Bit site) and it seems not to work—it stopped sending any data. I couldn't find any documentation to suggest that it would (or wouldn't), but I thought I'd try.

Perhaps we could combine the two tags? I've never tried that; do you have experience with it?

@chalin
Copy link
Author

chalin commented Nov 29, 2023

@erikbledsoe - right, I'm not surprised that you couldn't add more than one site tag.

Yes, I can connect other site tags to the main CNCF one. I used that feature during GA4 migration to link back to UA properties.

Which IDs would you like me to connect to?

@erikbledsoe
Copy link
Collaborator

@chalin
G-GJ6DCMDH4N

@chalin
Copy link
Author

chalin commented Nov 29, 2023

Done: connected. Can you try again? Thanks @erikbledsoe

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.

3 participants