Migrate doi pages to app router

[bklaing2]

Purpose

Migrate /doi.org pages to the app router

closes: #333

Approach

I followed a similar approach to the index page migration

Open Questions and Pre-Merge TODOs

• [ ] The most significant changes are in the src/app/doi folder.
  • Most of the other changes are due to imports changing to the new types.ts file and react-bootstrap wrappers, or adding the 'use client' directive
• [ ] When building the project, the CSS imports get mixed up. I moved the doi.css import from the root layout page into the styles.css file
• [x] Some cypress tests were throwing errors that don't appear when I manually follow the same steps. After spending some time looking into it with no success, I decided to disable them for the time being
  • This was resolved by
  • setting experimentalFetchPolyfill: false in cypress.config.ts
  • updating the GitHub action to test against the production build instead of the dev build
  • Telling Cypress to ignore hydration errors because Cypress and React Hydration don't get along
• [ ] Some react-bootstrap interactions (like the Tabs buttons on the DOI page) don't work when running yarn run dev. However, they work with yarn run build

Learning

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:

• Be humble in the language and feedback you give, ask don't tell.
• Consider using positive language as opposed to neutral when offering feedback. This is to avoid the negative bias that can occur with neutral language appearing negative.
• Offer suggestions on how to improve code e.g. simplification or expanding clarity.
• Ensure you give reasons for the changes you are proposing.

[cypress[bot]]

Passing run #1075 ↗︎

0

56

0

0

0

Details:

Merge 2f07793a2fe2b1fc3718ccba657ad60cc8a7094e into 27f23257ca87c04aac67098685a2...
Project: akita	Commit: 299df21661 ℹ️
Status: Passed	Duration: 03:43 💡
Started: Feb 29, 2024 9:56 AM	Ended: Feb 29, 2024 10:00 AM

Review all test suite changes for PR #342 ↗︎

[cypress[bot]]

3 flaky tests on run #1051 ↗︎

0

55

0

0

3

Details:

Fixed failing tests
Project: akita	Commit: 36216833af
Status: Passed	Duration: 02:56 💡
Started: Feb 8, 2024 3:32 PM	Ended: Feb 8, 2024 3:35 PM

  search.test.ts • 1 flaky test • Tests

View Output

Test		Artifacts
... > search with enter		Test Replay Screenshots

  personContainer.test.ts • 1 flaky test • Tests

View Output

Test		Artifacts
PersonContainer > identifiers		Test Replay Screenshots

  statistics.test.ts • 1 flaky test • Tests

View Output

Test		Artifacts
Overview > header		Test Replay Screenshots

Review all test suite changes for PR #342 ↗︎