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

Update to Dioxus 0.6 #1

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

pedropsouza
Copy link

First of all, thanks for the great crate!
I really wanted to use it for my 0.6.1 app, and the code seemed easy enough to port.

I did just blindly follow the migration guides for 0.4 -> 0.5 -> 0.6 and regrettably didn't notice the other 0.4 -> 0.5 fork until it was too late. That would've saved some work.

If there's any issue that keeps this from merging, let me know!

@feral-dot-io
Copy link
Owner

Thanks for taking the time to put together this PR.

I'm no longer using Dioxus so I can't accept this change. There's a certain level of commitment needed with making a release as I'd want to be prepared to fix bugs that come my way.

Would you be interested in taking over the project and crate? I think it has mostly reached its goal of making sortable tables easier and just needs to be kept up to date as Dioxus evolves. Work similar to this PR.

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