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 - Root Layout #1083

Open
boodland opened this issue Aug 19, 2024 · 2 comments
Open

App Router Migration - Root Layout #1083

boodland opened this issue Aug 19, 2024 · 2 comments
Assignees
Labels
complexity: advanced Time needed to do this ticket will be large e.g. 2-3+ days feature/enhancement New feature or request
Milestone

Comments

@boodland
Copy link
Contributor

boodland commented Aug 19, 2024

Sub issue of #702

App router uses a Root layout which is defined at the top of the app directory and applies to all routes. We need to migrate the contents of _app.tsx and _document.tsx to the root layout so they are applied for the app route.

Due to the complexity of the _app.tsx and _document.tsx files, the migration will be done according to the following steps as discussed in #1064:

  1. Creating the app directory and basic root layout with public testing page App router migration root layout #1085
  2. Add Newrelic configuration App router migration new relic configuration #1086
  3. Add GoogleTagManager script App router migration google tag manager script #1117
  4. Add Rollbar script App router migration rollbar script #1118
  5. Add OpenGraph Metadata App router migration opengraph metadata #1120
  6. Add ErrorBoundary component App router migration error boundary component #1121
  7. Add Material UI configuration App router migration material UI configuration #1122
  8. Add Analytics configuration App router migration analytics #1123

Step one will be the parent PR/branch where the rest of the steps will be created from and merged to, once all the steps are completed and merged the PR/branch can be merged within dev branch.

Store Provider, i18n, Shell layout, Crisp script, AuthGuard and pages migration will be tackled in another parent PR as discussed in #1064 due to conflict with pages router and/or dependencies on i18n.

Until all pages are migrated and _app.tsx and _document.tsx files are removed any changes in any of the logic migrated to the app router needs to be changed accordingly for the app router version as well.

@boodland
Copy link
Contributor Author

@kyleecodes, @eleanorreem or @annarhughes could you assign it to me please? Mentioning all of you so you are all aware of the state of the task.

Copy link
Contributor

Thank you @boodland you have been assigned this issue!
Please follow the directions in our Contributing Guide. We look forward to reviewing your pull request shortly ✨


Support Chayn's mission? ⭐ Please star this repo to help us find more contributors like you!
Learn more about Chayn here and explore our projects. 🌸

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
complexity: advanced Time needed to do this ticket will be large e.g. 2-3+ days feature/enhancement New feature or request
Projects
Development

No branches or pull requests

2 participants