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

pub export server is either from leptos_macro or leptos_spin_macro depending on if spin feature is enabled. #2280

Merged
merged 4 commits into from
Feb 15, 2024

Conversation

sjud
Copy link
Contributor

@sjud sjud commented Feb 8, 2024

This request add leptos_spin_macro and pub exports the server macro defined there if spin is a feature, it adds leptos_spin_macro as a dependency (based on benwis fork of it) but for this to work he needs to accept my PR there.

@benwis
Copy link
Contributor

benwis commented Feb 10, 2024

I like the PR, but I don't want to introduce a dependency on my fork of leptos-spin, so I think we can wait for them to release before we do this

@sjud
Copy link
Contributor Author

sjud commented Feb 13, 2024

I like the PR, but I don't want to introduce a dependency on my fork of leptos-spin, so I think we can wait for them to release before we do this

Sounds good to me!

@benwis benwis merged commit e33ee7e into leptos-rs:main Feb 15, 2024
7 of 60 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.

None yet

2 participants