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.

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

App router migration new relic configuration #1086

Open
wants to merge 18 commits into
base: next-js-migration
Choose a base branch
from

Conversation

boodland
Copy link
Contributor

Children PR for the progressive migration of _app.tsx and _document.tsx within the Root Layout

Issue link / number:
#1083

What changes did you make?
Created new relic configuration
Added new relic configuration to root layout

Why did you make the changes?
In order to be able to have new relic within app router

Did you run tests?
Yes

Navigate to http://localhost:3000/public-route-test to check that the head nr-browser-agent and body scripts are added properly

image

Copy link

vercel bot commented Aug 19, 2024

@boodland is attempting to deploy a commit to the Chayn Team on Vercel.

A member of the Team first needs to authorize it.

@boodland
Copy link
Contributor Author

@annarhughes This PR needs to point to app-router-migration branch once #1085 is merged

@boodland boodland force-pushed the app-router-migration-new-relic-configuration branch from c900116 to 7e53126 Compare August 20, 2024 08:06
@boodland boodland changed the base branch from develop to next-js-migration August 26, 2024 08:03
@boodland
Copy link
Contributor Author

@eleanorreem, @annarhughes I have just changed the base branch, have a look when possible. Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant