-
Notifications
You must be signed in to change notification settings - Fork 18
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
Extend tool to do simple manual CLI driven updates for example of TextCards #7
Comments
Hi @jornh , good thing you asked first :) |
Thanks for the superfast feedback! I can understand your reasoning. I agree, it is a simple API where you can quickly build some one-off solution in any language. I was more thinking about the more people are spread across different home-grown one-off's there's also more each have to maintain over time on their own when the API changes like your #6. It's not just the sharing or not of code it's also the sharing of issue tracker etc. Anyway, I in fact don't have an urgent need myself, just thought it could be a fun one-night hack. I'll think about the options going forward. Thanks! |
Yep, what you describe here sounds like a spin-off project extracting the API classes in this repo to a separate library and maintained independently of the concrete use case of exporting/importing Metabase state (the scope of this project). Unfortunately I wouldn't be able to afford maintaining such a project, but if anyone wants to do it they can (just added a LICENSE file). Also for the Metabase team it would be useful to track client libraries like that somewhere in the documentation I think. |
To give an example of the use case - see the topic Programmatic way to update Dashboard Text Card
Is this something that you would consider being part of
elevate.metabase.tools
?It looks like it could easily be built on top of for example:
elevate.metabase.tools/metabase-exporter/MetabaseApi.cs
Line 39 in dcca4c8
How would you prefer it done. Maybe as separate
.exe
?I'm considering to roll up my C# sleeves and submit a PR - but just wanted to get your opininion on this first
The text was updated successfully, but these errors were encountered: