Skip to content

Conversation

@brenelz
Copy link
Contributor

@brenelz brenelz commented Jan 30, 2026

Main goals:

  • Update tanstack/react-router/start
  • Update react/react-dom

Chores

  • Fix typescript errors showing on commit (fixed some manually and some using opus)
  • Fix as many eslint errors as possible (fixed some manually and some using opus)

@netlify
Copy link

netlify bot commented Jan 30, 2026

👷 Deploy request for tanstack pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit cb826e1

params: { libraryId, version } as never,
})
}
return undefined as never
Copy link
Contributor Author

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",
Copy link
Contributor Author

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",
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

upgrade react

to: './docs',
params: { libraryId: library.id },
to: '/$libraryId/$version/docs',
params: { libraryId: library.id, version: 'latest' },
Copy link
Contributor

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?

Copy link
Contributor Author

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?

to: './docs',
params: { libraryId: library.id },
to: '/$libraryId/$version/docs',
params: { libraryId: library.id, version: 'latest' },
Copy link
Contributor

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' },
Copy link
Contributor

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' },
Copy link
Contributor

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

Copy link
Contributor Author

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

<Link
to="/showcase"
search={{
page: 1,
Copy link
Contributor

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?

Copy link
Contributor Author

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants