Closed bklaing2 closed 5 months ago
0 | 64 | 0 | 0 | 1 |
Details:
Merge 8f84074792a4eb32a690f1346f825d1b7e0c3b84 into cb3320a0268301f56d24db05a944... | |||
Project: akita | Commit: b6b476349f ℹ️ |
||
Status: Passed | Duration: 02:44 💡 | ||
Started: Feb 7, 2024 10:57 AM | Ended: Feb 7, 2024 11:00 AM |
View
Output
Test | Artifacts | |
---|---|---|
Overview > header |
Test Replay
Screenshots
|
Purpose
Migrate the current index page from the
pages
router to theapp
routercloses: Add github issue that originated this PR
Approach
Open Questions and Pre-Merge TODOs
Server
version in cases that they were using features supported only in thepages
router (e.g.useRouter
). These are meant to be temporary until all of the pages are migratedreact-bootstrap
isn't supported in server components. I created a'use client'
wrapper for theGrid
,Row
,Col
, andAlert
components because they are commonly used in server components.<script>
tag into<head>
when it runs it's tests, leading to hydration errors. I wrapped the contents of<head>
in a<Suspense>
boundary per this commentSource Sans Pro
<link>
in favor of importing usingnext/font/google
. Of course, it is now calledSource Sans 3
, so I had to remove references toSource Sans Pro
indoi.css
next-plausible
, and moved itsProvider
into the<head>
per its documentationLearning
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: