-
Notifications
You must be signed in to change notification settings - Fork 201
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
Bring back Excel support #281
Comments
Two bits of feedback:
|
The plugin architecture is meant to keep datasources decoupled from the main repository, so 99% this will be implemented as a plugin. That said, you won't need additional tooling nor network connectivity. Plugins can be installed using OctoSQL itself and are then available locally |
Got it on the tooling bit, but the connectivity thing seems contradictory? I imagine that
Sure, that works.
(Forgive me, I have some machines with interesting network setups that make me hate magical package managers and plugin installers.) |
Sorry, what I meant is that you only need it once, on setup.
The folder structure is bit more interesting than that, but yeah, basically this. |
Gotcha, yeah sometimes I just want to move the binary to another network and run it somewhere I don't have connectivity.
Awesome. If you have or could make a quick reference on how to set this up, that would be fantastic. (Not needed until after Excel support is added.) Thanks for your all your work! |
+1natively without requiring plugins; I just want to be able to download octosql and run it without additional tooling, network connectivity, etc. |
Excel support would be awesome, we now have to convert to csv first before we can query. It would save that step. Excel support also allows us to automate excel to csv conversions in other use cases. Plugin install seems straightforward so no issue with that. |
It got lost during the rewrite, should be recreated as a plugin.
Should mostly be a copy-paste of the now-deleted excel datasource, as mentioned in #135
The text was updated successfully, but these errors were encountered: