-
-
Notifications
You must be signed in to change notification settings - Fork 286
WIP: chore: update tanstack start #690
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
base: main
Are you sure you want to change the base?
Conversation
👷 Deploy request for tanstack pending review.Visit the deploys page to approve it
|
| params: { libraryId, version } as never, | ||
| }) | ||
| } | ||
| return undefined as never |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
hmm this seems a bit weird
| "@tanstack/react-router": "1.157.16", | ||
| "@tanstack/react-router-devtools": "1.157.16", | ||
| "@tanstack/react-router-ssr-query": "1.157.16", | ||
| "@tanstack/react-start": "1.157.16", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
upgrade react router
| "react": "^19.2.4", | ||
| "react-colorful": "^5.6.1", | ||
| "react-dom": "^19.2.0", | ||
| "react-dom": "^19.2.4", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
upgrade react
src/components/landing/DbLanding.tsx
Outdated
| to: './docs', | ||
| params: { libraryId: library.id }, | ||
| to: '/$libraryId/$version/docs', | ||
| params: { libraryId: library.id, version: 'latest' }, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why is "latest" added here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yeah that is odd doesn't look like it needs it. The version will just default to latest?
src/components/landing/DbLanding.tsx
Outdated
| to: './docs', | ||
| params: { libraryId: library.id }, | ||
| to: '/$libraryId/$version/docs', | ||
| params: { libraryId: library.id, version: 'latest' }, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why is "latest" added here?
| to: './docs', | ||
| params: { libraryId: library.id }, | ||
| to: '/$libraryId/$version/docs', | ||
| params: { libraryId: library.id, version: 'latest' }, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why latest?
| to: './docs', | ||
| params: { libraryId: library.id }, | ||
| to: '/$libraryId/$version/docs', | ||
| params: { libraryId: library.id, version: 'latest' }, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i see this happened a lot, wont repeat this comment all over :D
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I removed this in the latest commit
src/components/ShowcaseSection.tsx
Outdated
| <Link | ||
| to="/showcase" | ||
| search={{ | ||
| page: 1, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
shouldnt this be set by the routes validator?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated the searchSchema's instead of the links in some cases
Main goals:
Chores