🔧 Enable GTM and CookieConsent for production #575
Conversation
|
sasamuku
requested review from
hoshinotsuyoshi,
FunamaYukina,
junkisai and
MH4GF
and removed request for
a team
|
Note I would like to check if the tag are delivered correctly, so let me merge by myself. |
| @@ -51,7 +51,7 @@ export default function ERDViewer({ | |||
| errorObjects={errorObjects} | |||
| /> | |||
| </VersionProvider> | |||
| {process.env.NEXT_PUBLIC_ENV_NAME !== 'production' && | |||
| {process.env.NEXT_PUBLIC_ENV_NAME === 'production' && | |||
There was a problem hiding this comment.
Q: Is there any reason not to display it in a preview or develpment environment?
There was a problem hiding this comment.
The preview URL changes every time, so I thought Accept cookie might be noisy.
There was a problem hiding this comment.
However, it might be a good idea to make the tags ready to be accepted, and sent local or preview as well 🤔
Let me fix this.
There was a problem hiding this comment.
@sasamuku
Could you implement the same approach as in this PR to pass the appEnv information to GTM in ERD Web? 🙏
Summary
Enable GTM and CookieConsent for production.
Note
I would like to check if the tag are delivered correctly, so let me merge by myself.
Related Issue
Changes
Testing
Other Information