Skip to content
This repository has been archived by the owner on Sep 22, 2023. It is now read-only.

AMP-83919 [BigQueryGA4Import] Add notes of BigQuery import to all places mentioning GA4 #960

Merged
merged 5 commits into from
Sep 7, 2023

Conversation

bh202205
Copy link
Collaborator

@bh202205 bh202205 commented Sep 6, 2023

Amplitude Developer Docs PR

Description

This PR adds notes of BigQuery import to all places mentioning GA4 to track customers/users attention for our support of GA4 import via BigQuery import.

Change type

  • Doc bug fix. Fixes #[insert issue number]. Amplitude contributors include Jira issue number.
  • Doc update.
  • New documentation.
  • Non-documentation related fix or update.

PR checklist:

  • My documentation follows the style guidelines of this project.
  • I previewed my documentation on a local server using mkdocs serve.
  • Running mkdocs serve didn't generate any failures.
  • I have performed a self-review of my own documentation.

@amplitude-dev-docs

@github-actions
Copy link

github-actions bot commented Sep 6, 2023

🦙 MegaLinter status: ✅ SUCCESS

Descriptor Linter Files Fixed Errors Elapsed time
✅ MARKDOWN markdownlint 4 0 0.39s

See detailed report in MegaLinter reports
Set VALIDATE_ALL_CODEBASE: true in mega-linter.yml to validate all sources, not only the diff

MegaLinter is graciously provided by OX Security

@aws-amplify-us-east-1
Copy link

This pull request is automatically being deployed by Amplify Hosting (learn more).

Access this pull request here: https://pr-960.d19s7xzcva2mw3.amplifyapp.com

Copy link
Contributor

@joez2 joez2 left a comment

Choose a reason for hiding this comment

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

Previewed the change. LGTM. Thanks for updating the dev docs!

Copy link
Contributor

@markzegarelli markzegarelli left a comment

Choose a reason for hiding this comment

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

Thanks @bh202205!

I edited the content to align with our style elsewhere.

One question I have is, "BigQuery Import for GA4 Beta"...does this mean that we are working on a beta of the BigQuery Import feature? Or will the BigQuery Import feature work on a beta version of GA4?

docs/data/destinations/google-analytics-4-firebase.md Outdated Show resolved Hide resolved
docs/data/destinations/google-analytics-4-gtag.md Outdated Show resolved Hide resolved
@bh202205
Copy link
Collaborator Author

bh202205 commented Sep 7, 2023

Thanks @bh202205!

I edited the content to align with our style elsewhere.

One question I have is, "BigQuery Import for GA4 Beta"...does this mean that we are working on a beta of the BigQuery Import feature? Or will the BigQuery Import feature work on a beta version of GA4?

@markzegarelli Thank you for all your comments and just updated the files!

To answer your question, let me give you some context. BigQuery is one of the popular data warehouses that our customers use and it's a google cloud product. We have a data warehouse integration called: BigQuery Import, which allows customers to import data from their BigQuery. GA4 (google analytics 4) is one of our competitors and what we are building is an kind of new integration that allows customers to import their GA4 data to Amplitude based on our existing BigQuery Import.

Let me know if this makes sense to you. Thank again!

@bh202205 bh202205 merged commit cdce90c into main Sep 7, 2023
4 checks passed
@bh202205 bh202205 deleted the AMP-83919-bigquery-ga4-import branch September 7, 2023 21:01
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants