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

docs(examples/google-analytics): Change to getting GA_TRACKING_ID via .env and update README #2960

Merged
merged 2 commits into from
Apr 29, 2022

Conversation

chiangs
Copy link
Contributor

@chiangs chiangs commented Apr 22, 2022

This PR updates the Google Analytics example. The example prior to this PR attempted to import the GA_TRACKING_ID in the client util but the value won't be available when attempting to directly use it in the render.

The example in this PR has the GA_TRACKING_ID available in the process.env. Then the loader in roots.tsx provides and returned via useLoaderData.

@remix-cla-bot
Copy link
Contributor

remix-cla-bot bot commented Apr 22, 2022

Hi @chiangs,

Welcome, and thank you for contributing to Remix!

Before we consider your pull request, we ask that you sign our Contributor License Agreement (CLA). We require this only once.

You may review the CLA and sign it by adding your name to contributors.yml.

Once the CLA is signed, the CLA Signed label will be added to the pull request.

If you have already signed the CLA and received this response in error, or if you have any questions, please contact us at [email protected].

Thanks!

- The Remix team

@remix-cla-bot
Copy link
Contributor

remix-cla-bot bot commented Apr 22, 2022

Thank you for signing the Contributor License Agreement. Let's get this merged! 🥳

@chiangs chiangs changed the title Change to getting GA_TRACKING_ID via .env and update README docs(examples) Google Analytics: Change to getting GA_TRACKING_ID via .env and update README Apr 23, 2022
@chiangs chiangs changed the title docs(examples) Google Analytics: Change to getting GA_TRACKING_ID via .env and update README docs(examples/google-analytics): Change to getting GA_TRACKING_ID via .env and update README Apr 23, 2022
@chaance chaance merged commit 777d70c into remix-run:main Apr 29, 2022
aaronpowell pushed a commit to aaronpowell/remix that referenced this pull request May 10, 2022
aaronpowell pushed a commit to aaronpowell/remix that referenced this pull request May 12, 2022
aaronpowell pushed a commit to aaronpowell/remix that referenced this pull request May 15, 2022
aaronpowell pushed a commit to aaronpowell/remix that referenced this pull request May 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants