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.

Sign up for GitHub

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

Jump to bottom

migrate to AppRouter #216

Open
wants to merge 10 commits into
base: main
Choose a base branch
from
Open

migrate to AppRouter #216

wants to merge 10 commits into from

Conversation

CrossEvol
Copy link

Description of what you did:

  • update orca client from PageRouter to AppRouter
  • fix some styled-component errors
  • supplement the types for Dispatch and backend API and backend models
  • provide pnpm workspaces setting

My PR is a:

  • 🐛 Bug fix closes #number
  • 💅 Enhancement closes #number
  • 🚀 New feature closes #number

Main update on the:

  • orca-api
  • orca-frontend

@CrossEvol
use pnpm workspaces as monorepo
d7c4b66
@CrossEvol
migrate from PageRouter to AppRouter
eb9b788
@CrossEvol
update dependencies except mongoose
11fdd0a
@CrossEvol
remove .babelrc && defaultProps
8c95e06
@CrossEvol
<a> can not nested, update style.ts for <Link/>
3d959e8
@CrossEvol
remove all the warnings lik e
92da715 
`Warning: React does not recognize the `hideIcon` prop on a DOM element. If you intentionally want it to appear in the DOM as a custom attribute, spell it as lowercase `hideicon` instead. If you accidentally passed it from a parent component, remove it from the DOM element.`

by use
`import shouldForwardProp from '@styled-system/should-forward-prop';
`
@CrossEvol
type definition optimization for orca-api
7938571
@CrossEvol
migrate .eslintrc to eslint.config.mjs and pnpm run lint in orca-api
c0758f2
@CrossEvol
degrade eslint from v9 to v8, for eslint-config-next did not support v9
66b3dee
@CrossEvol
pnpm run ts:check, mainly deal with Dispatch<UnknownActions>
78d2e5d
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@CrossEvol