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: leptos_router::params_map! #1973

Merged
merged 1 commit into from
Nov 2, 2023
Merged

fix: leptos_router::params_map! #1973

merged 1 commit into from
Nov 2, 2023

Conversation

chrisp60
Copy link
Contributor

@chrisp60 chrisp60 commented Nov 2, 2023

fixes #1971 through reimplementation.

Comes with the benefit of knocking a crate out of the deps tree.

Fixing implementation comes with the benefit of knocking a crate out of
the deps tree (`common_macros`).
@gbj
Copy link
Collaborator

gbj commented Nov 2, 2023

Awesome! Thank you so much.

@gbj gbj merged commit 3adfd33 into leptos-rs:main Nov 2, 2023
@chrisp60 chrisp60 deleted the fix-params-map-macro branch November 2, 2023 22:15
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.

leptos_router::params_map! remove from public API
2 participants