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.

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

App router migration rollbar script #1118

Merged
merged 13 commits into from
Sep 20, 2024
+10 −3
Merged

App router migration rollbar script #1118

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
440bcb0
Add basic Root Layout
boodland Aug 19, 2024
78f94ae
Add metadata
boodland Aug 19, 2024
e4bffac
Add reference included by next
boodland Aug 19, 2024
844f412
Add comment to explain the default meta tags
boodland Aug 19, 2024
d857356
Add public testing page to check the migration
boodland Aug 19, 2024
fbc9686
Merge branch 'next-js-migration' into app-router-migration-root-layout
boodland Aug 30, 2024
86241af
Remove non required file
boodland Aug 30, 2024
a32b52a
Use script tag and set id
boodland Sep 3, 2024
69bf65d
Add rollbar script to root layout
boodland Sep 3, 2024
ff18731
Make it beforeInteractive so it is included in the head
boodland Sep 3, 2024
2e2560f
Move to _app as next/script is not working in _document
boodland Sep 3, 2024
5087524
Merge branch 'next-js-migration' into app-router-migration-rollbar-sc…
boodland Sep 12, 2024
49f3915
Merge branch 'next-js-migration' into app-router-migration-rollbar-sc…
boodland Sep 18, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 2 additions & 0 deletions app/layout.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,5 +1,6 @@
import { Metadata } from 'next';
import GoogleTagManagerScript from '../components/head/GoogleTagManagerScript';
import RollbarScript from '../components/head/RollbarScript';

// Nextjs automatically includes for each route two default meta tags, charset and viewport
// https://nextjs.org/docs/app/building-your-application/optimizing/metadata#default-fields
Expand All @@ -22,6 +23,7 @@ export default function RootLayout({
but sending an event using sendGTMEvent requires an object rather than a list of arguments so the current gtag api function would need to be adapted
*/}
<GoogleTagManagerScript />
<RollbarScript />
{children}
</body>
</html>
Expand Down
7 changes: 6 additions & 1 deletion components/head/RollbarScript.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,6 +1,11 @@
import Script from 'next/script';

const RollbarScript = () => {
return (
<script
// eslint-disable-next-line @next/next/no-before-interactive-script-outside-document
<Script
id="rollbar"
strategy="beforeInteractive"
dangerouslySetInnerHTML={{
__html: `
var _rollbarConfig = {
Expand Down
2 changes: 2 additions & 0 deletions pages/_app.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,7 @@ import { useEffect } from 'react';
import { Provider } from 'react-redux';
import CrispScript from '../components/crisp/CrispScript';
import GoogleTagManagerScript from '../components/head/GoogleTagManagerScript';
import RollbarScript from '../components/head/RollbarScript';
import Consent from '../components/layout/Consent';
import ErrorBoundary from '../components/layout/ErrorBoundary';
import Footer from '../components/layout/Footer';
Expand Down Expand Up @@ -83,6 +84,7 @@ function MyApp(props: MyAppProps) {
<meta name="viewport" content="initial-scale=1, width=device-width" />
</Head>
<GoogleTagManagerScript />
<RollbarScript />
<CrispScript />
<ThemeProvider theme={theme}>
<CssBaseline />
Expand Down
2 changes: 0 additions & 2 deletions pages/_document.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,6 @@ import { AppType } from 'next/app';
import Document, { Head, Html, Main, NextScript } from 'next/document';
import * as React from 'react';
import OpenGraphMetadata from '../components/head/OpenGraphMetadata';
import RollbarScript from '../components/head/RollbarScript';
import createEmotionCache from '../config/emotionCache';
import { MyAppProps } from './_app';

Expand All @@ -26,7 +25,6 @@ export default class MyDocument extends Document<NewRelicProps> {
rel="stylesheet"
/>
<OpenGraphMetadata />
<RollbarScript />
</Head>
<body>
<Main />
Expand Down
Loading