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

UserModel has no way to save to xlsx. #262

Closed
zaphar opened this issue Jan 30, 2025 · 3 comments · Fixed by #263
Closed

UserModel has no way to save to xlsx. #262

zaphar opened this issue Jan 30, 2025 · 3 comments · Fixed by #263

Comments

@zaphar
Copy link

zaphar commented Jan 30, 2025

The Model can be passed to the functions save_to_* to save to xlsx. The UserModel consumes the Model however and has no corresponding mechanism to save to xlsx.

I have considered adopting the UserModel in my own terminal spreadhseet app sheetsui but currently there is no way to save to a file if you use it.

@nhatcher
Copy link
Member

Hi @zaphar, first of all awesome job with sheetui!

Let me know if #263 fixes it for you.

We can't really save_to_xlsx in the use model itself because that would bring the whole xlsx crate dependency that we don't want to have in the wasm build

@zaphar
Copy link
Author

zaphar commented Jan 30, 2025

@nhatcher, I'll try to test that later this evening but it looks like it should be sufficient. Thanks for the kind words on sheetui.

@zaphar
Copy link
Author

zaphar commented Jan 31, 2025

@nhatcher that seems to work as expected, thanks!

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 a pull request may close this issue.

2 participants