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
Connects to #106. Google Analytics tag update. #118
Conversation
Also fixed bug in which wrong release version number is passed to Google Analytics data download tracking event
…site was launched
…rors cause problems while compiling and building the app ESLint errors didn't cause any issues prior to react-scripts v3.x. So the React team released this patch in response to that. Also upgraded the storybook dependencies to address the vulnerability issue in its peer dependency.
…using app build problem in react-scripts v4.x
…or CRA production build This workaround could be temporary and reverted in the future
@jimmyzhen Regarding step 5, can you please let me know what line are you referring to in the |
@ysyskim4 Thank you for reviewing this PR. I will share a screenshot to explain what I meant. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great. Thanks.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Everything works as described in the test steps.
Key Changes
analytics.js
withgtag.js
react-scripts
dependency to latest version to address issue in which ESLint errors prevent app from compiling indevelopment
environment, as well as a vulnerability issue associated with its peer dependencyimmer
storybook
dependencies to address a vulnerability issue associated with its peer dependencyimmer
Technical Note
Use
nvm
to setnode.js
tov14.15.4
Steps to test
git clone
the repo and runyarn install
at the repo's root directory..env
file and addESLINT_NO_DEV_ERRORS=true
in the file (for references, see https://github.com/facebook/create-react-app/blob/master/docusaurus/docs/advanced-configuration.md and Upgrade eslint-webpack-plugin to fix opt-out flag facebook/create-react-app#10590).yarn inspect-branch
, and expect to see ESLint warnings in the console (both in Terminal and browser) while the app can be successfully compiled and run.yarn audit
and expect to see0 vulnerabilities found
in the output message.*.jsx
files (e.g.releaseEntry.jsx
) and expect to see some lines of code underlined (red wavy lines) - an indication thateslint
andprettier
are hinting fixes.dev
property.Realtime > Content
. Then try clicking through different pages on the local instance of the frontend app. Expect to see the pageviews being reported in real time in Google Analytics.Realtime > Events
. On the homepage of the local instance of the frontend app, try clicking on the 2 links. Expect to see the events being reported in real time in Google Analytics.