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.

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

add deprecation warnings to RR v6 and Remix v2 #11648

Closed
ryanflorence opened this issue Jun 10, 2024 · 10 comments · Fixed by #11750
Closed

add deprecation warnings to RR v6 and Remix v2 #11648

ryanflorence opened this issue Jun 10, 2024 · 10 comments · Fixed by #11750
Assignees

Comments

@ryanflorence
Copy link
Member

No description provided.

@brophdawg11 brophdawg11 self-assigned this Jun 28, 2024
@brophdawg11
Copy link
Contributor

#11750

@brophdawg11 brophdawg11 linked a pull request Jun 28, 2024 that will close this issue
@brophdawg11 brophdawg11 changed the title add warnings for v7 flags in v6 add deprecation warnings to RR v6 and Remix v2 Oct 9, 2024
@brophdawg11
Copy link
Contributor

  • Add warnings for stable future flags
  • Add deprecation warnings to APIs - json/defer in Remix
    • Probably can't warn from the utils but can detect the type of response returned from loader/action and warn there?
  • Remove json/defer in Rr v7
  • Update redirect utils in v7 to use data under the hood

@brophdawg11
Copy link
Contributor

remix-run/remix#10126

@brophdawg11
Copy link
Contributor

#12144

Copy link
Contributor

🤖 Hello there,

We just published version 6.28.0-pre.0 which involves this issue. If you'd like to take it for a test run please try it out and let us know what you think!

Thanks!

Copy link
Contributor

github-actions bot commented Nov 6, 2024

🤖 Hello there,

We just published version 6.28.0 which involves this issue. If you'd like to take it for a test run please try it out and let us know what you think!

Thanks!

Copy link
Contributor

🤖 Hello there,

We just published version 7.0.0 which involves this issue. If you'd like to take it for a test run please try it out and let us know what you think!

Thanks!

Copy link
Contributor

🤖 Hello there,

We just published version 6.28.1-pre.0 which involves this issue. If you'd like to take it for a test run please try it out and let us know what you think!

Thanks!

Copy link
Contributor

🤖 Hello there,

We just published version 6.28.1 which involves this issue. If you'd like to take it for a test run please try it out and let us know what you think!

Thanks!

Copy link
Contributor

🤖 Hello there,

We just published version 6.28.2-pre.0 which involves this issue. If you'd like to take it for a test run please try it out and let us know what you think!

Thanks!

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 a pull request may close this issue.

2 participants