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

Space (or any non-encoded character) in route segment causes a hydration mismatch in Form action #5599

Closed
1 task done
frontsideair opened this issue Feb 28, 2023 · 2 comments
Assignees
Labels
bug Something isn't working hydration

Comments

@frontsideair
Copy link
Contributor

What version of Remix are you using?

1.13.0

Are all your remix dependencies & dev-dependencies using the same version?

  • Yes

Steps to Reproduce

  1. Use space in a route segment e.g. /post/hello world
  2. Use a Form in that route

Expected Behavior

The encoding or url should match on the client and the server and there should be no warning.

Actual Behavior

I get a hydration warning Warning: Prop action did not match. Server: "/post/hello world" Client: "/hello%20world"

@github-actions
Copy link
Contributor

This issue has been automatically marked stale because we haven't received a response from the original author in a while 🙈. This automation helps keep the issue tracker clean from issues that are not actionable. Please reach out if you have more information for us or you think this issue shouldn't be closed! 🙂 If you don't do so within 7 days, this issue will be automatically closed.

@github-actions github-actions bot added the needs-response We need a response from the original author about this issue/PR label Apr 29, 2023
@machour machour removed the needs-response We need a response from the original author about this issue/PR label Apr 29, 2023
@brophdawg11 brophdawg11 self-assigned this Aug 3, 2023
@brophdawg11 brophdawg11 added bug Something isn't working and removed bug:unverified labels Aug 8, 2023
@brophdawg11
Copy link
Contributor

This is resolved by remix-run/react-router#10769 and should be available in Remix v2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working hydration
Projects
No open projects
Status: Merged
Development

No branches or pull requests

3 participants