Skip to content

Remove Uri from responders - replace with String#32

Merged
robertwayne merged 2 commits intorobertwayne:mainfrom
skwee357:main
Jun 3, 2025
Merged

Remove Uri from responders - replace with String#32
robertwayne merged 2 commits intorobertwayne:mainfrom
skwee357:main

Conversation

@skwee357
Copy link
Contributor

@skwee357 skwee357 commented Jun 2, 2025

This is a PR for #31

I have decided to leave HxCurrentUrl with Uri dependency, as this is aligned with how axum extracts the current Uri. I have removed Uri dependency from all Url/Location headers.

For more information, see hyper Uri does not support #fragment

Changes

  • Replace Uri with String for HxLocation responder
  • Replace Uri with String for HxPushUrl responder
  • Replace Uri with String for HxRedirect responder
  • Replace Uri with String for HxReplaceUrl responder

@robertwayne robertwayne merged commit 4eae8b3 into robertwayne:main Jun 3, 2025
2 checks passed
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