airjp73 / rvf

Easy form validation and state management for React and Remix
https://rvf-js.io
MIT License
840 stars 66 forks source link

[Bug]: useTransition deprecated in favor of useNavigation #261

Closed justinwaite closed 1 year ago

justinwaite commented 1 year ago

Which packages are impacted?

What version of these packages are you using?

Please provide a link to a minimal reproduction of the issue.

N/A

Steps to Reproduce the Bug or Issue

Navigate to a page that uses ValidatedForm

Expected behavior

No warnings should be in the console. Instead, the following is shown:

react_devtools_backend.js:2655 ⚠️ DEPRECATED: The `useTransition` hook has been deprecated in favor of `useNavigation` and will be removed in Remix v2.  Please update your code to leverage `useNavigation`.

See https://remix.run/docs/hooks/use-transition and https://remix.run/docs/hooks/use-navigation for more information. 
    at ValidatedForm (http://localhost:3000/cod-v2/build/_shared/chunk-FVKP76MZ.js:2478:3)

Screenshots or Videos

No response

Platform

Additional context

No response

airjp73 commented 1 year ago

Thanks for the callout! Addressing this will require us to increase the minimum supported remix version, so we'll need to do a major version bump.

Have the team given any estimate of when v2 might be released? If there are any other API changes we need to update for v2 down the road, I'd likely want to include all of them in the same "v2 support" release.

Definitely want to fix this, but also want to err on the side of taking my time with it since it's currently non-urgent.

kristremblay commented 1 year ago

@airjp73 This might be of interest to you https://remix.run/blog/future-flags. Cheers!

justinwaite commented 1 year ago

@airjp73 Remix 1.15.0 officially deprecates the rest of the v1 APIs in favor of v2, so it looks like we're getting closer to v2.

airjp73 commented 1 year ago

Fixed in #284

Hoping to cut the v5 release sometime later today.