fix(gtm): move noscript under body

[mazipan]

Closes issue on #419

Description

Move GTM under the body

Current Tasks

• Move GTM to be precise under the body
• Move the noscript from _app to _document

[netlify]

✔️ Deploy Preview for wargabantuwarga ready!

🔨 Explore the source changes: 124dffd

🔍 Inspect the deploy log: https://app.netlify.com/sites/wargabantuwarga/deploys/60fe3bb7a5fd700007cde716

😎 Browse the preview: https://deploy-preview-438--wargabantuwarga.netlify.app

[codecov]

Codecov Report

Merging #438 (124dffd) into main (304f0cd) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #438   +/-   ##
=======================================
  Coverage   39.58%   39.58%           
=======================================
  Files          67       67           
  Lines         864      864           
  Branches      257      257           
=======================================
  Hits          342      342           
  Misses        516      516           
  Partials        6        6           

Impacted Files	Coverage Δ
pages/_app.tsx	0.00% <ø> (ø)
pages/_document.tsx	0.00% <ø> (ø)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 304f0cd...124dffd. Read the comment docs.

[zainfathoni]

What do you think, @resir014?
With all these caveats, do you think it's okay to move it out from _app to _document?

[mazipan]

I think we need to differentiate ya, the <Script /> should be keep on _app, since it have a logic.
But for the noscript, it should be fine

[zainfathoni]

LGTM 💯

Thanks, Mas @mazipan! 🙏

[mazipan]

Do not merge it now ya.
I need to double check on the Netlify Preview 🙏

[mazipan]

The <Script/> can create a multiple GTM script :-(

[zainfathoni]

The <Script/> can create a multiple GTM script :-(

Maybe try replacing my Script implementation with next/script 😅
https://nextjs.org/docs/basic-features/script

But remember not to put it in the _document.tsx file

[resir014]

@zainfathoni Yes, let's replace everything with next/script in a separate PR.

[resir014]

Issue created: #439