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

fix(router): fix loader request method #9660

Merged
merged 2 commits into from
Nov 30, 2022

Conversation

brophdawg11
Copy link
Contributor

Loader revalidation requests after an action were being passed the same request used for the action, which was still a POST with formData. Loaders should be receiving a GET request without the body

@changeset-bot
Copy link

changeset-bot bot commented Nov 30, 2022

🦋 Changeset detected

Latest commit: 7264d92

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 5 packages
Name Type
@remix-run/router Patch
react-router Patch
react-router-dom Patch
react-router-dom-v5-compat Patch
react-router-native Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@ryanflorence ryanflorence merged commit 80bd977 into dev Nov 30, 2022
@ryanflorence ryanflorence deleted the brophdawg11/fix-loader-request branch November 30, 2022 21:17
@MichaelDeBoey MichaelDeBoey changed the title Fix loader request method fix(router): fix loader request method Dec 3, 2022
brophdawg11 added a commit that referenced this pull request Dec 5, 2022
* Fix loader request method

* add changeset
brophdawg11 added a commit that referenced this pull request Dec 5, 2022
* Fix loader request method

* add changeset
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants