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.

Sign up for GitHub

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

Jump to bottom

Gtm container snippet added #678

Merged

alex-litvak merged 3 commits into gh-pages from gtm-tags

Nov 19, 2021

Contributor

jfaMan commented Nov 10, 2021

Hi, Alex. Could you review this please?

Added GTM container snippet to head.html
Created partial for the body snippet and added it to overview.html and default.html (they seem to be the two index pages that require it). Please tell me if this is fine.

Styling

Added missing indentation on both pages for clarity.
Changed naming of 'tag' to 'container' for clarity since it's a container snippet

Allan Szewczyk added 3 commits

November 10, 2021 14:36


          added gtm tag to head

692893a


          created gtm body tag partial and added to overview and default pages

fc9f207


          changed gtm tag to gtm container for clarity

755d5e4

alex-litvak merged commit 009cbe6 into gh-pages

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet