Closed bklaing2 closed 5 months ago
![]() |
![]() |
![]() |
![]() |
![]() |
Details:
Merge 30b3d371f1bd208ce36d3c9c3f57a36bbe7a8968 into bfdbdd05ef81edbeb9b2e22b4e21... | |||
Project: akita | Commit: 03fff29392 ℹ️ |
||
Status: Passed | Duration: 02:39 💡 | ||
Started: Feb 1, 2024 3:42 PM | Ended: Feb 1, 2024 3:44 PM |
View
Output
Test | Artifacts | |
---|---|---|
PersonContainer > tags |
Test Replay
Screenshots
|
Purpose
Set up the layout for the app router
closes: #332
Approach
Migrated the
Layout
component being used across thepages
router to work with theapp
router. It now lives in thesrc/app/layout.tsx
file, and any sibling or childpage.tsx
files are passed to it as children.Open Questions and Pre-Merge TODOs
app
folder404.tsx
file in thepages
directory because the 404 page was throwing an error.pages
andapp
routerapp
routerLearning
Referenced the NextJS docs to understand the app router.
Types of changes
[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
Reviewer, please remember our guidelines: