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

feat(project): add meta tag for GA #330

Merged
merged 6 commits into from
Jul 6, 2023
Merged

Conversation

olga-jwp
Copy link
Collaborator

@olga-jwp olga-jwp commented Jul 5, 2023

Description

Add meta tag for GA

This PR solves BC-570

Steps completed:

  • Upgrade vite version
  • Add changes to createHtmlPlugin()
  • Set variable for GA in .env

According to our definition of done, I have completed the following steps:

  • Acceptance criteria met
  • Unit tests added
  • Docs updated (including config and env variables)
  • Translations added
  • UX tested
  • Browsers / platforms tested
  • Rebased & ready to merge without conflicts
  • Reviewed own code

.env Outdated
@@ -6,3 +6,5 @@ APP_DEFAULT_LANGUAGE=en

# a comma separated list of languages that are enabled (this only works for languages that are enabled in the app)
APP_ENABLED_LANGUAGES=en,es

APP_GOOGLE_SITE_VERIFICATION_ID=joKVm0-rBn0Xqsqrdu_MPgQU2_jJvKSnqwEwodBvnhk
Copy link
Contributor

Choose a reason for hiding this comment

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

Let's not include it in the .env file (because that will get picked up by any forked repos) and instead still feed it in via vars and env in the github action. It's not a sensitive value, but nice to implement in a way that other developers can fork and set their own values without create git noise.

@github-actions
Copy link

github-actions bot commented Jul 5, 2023

Visit the preview URL for this PR (updated for commit 0ee3686):

https://ottwebapp--pr330-feat-add-meta-tag-fo-8f9ud6mf.web.app

(expires Sat, 05 Aug 2023 11:35:19 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: c198f8a3a199ba8747819f7f1e45cf602b777529

Copy link
Contributor

@dbudzins dbudzins left a comment

Choose a reason for hiding this comment

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

That inject is cool! Nice work.

@dbudzins dbudzins merged commit 87db5f9 into develop Jul 6, 2023
@olga-jwp olga-jwp deleted the feat/add-meta-tag-for-GA branch January 3, 2024 15:25
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.

2 participants