[GitHub] [superset] villebro commented on pull request #14628: feat: Export Chart to .xml format

[GitBox <gi...@apache.org>]

villebro commented on pull request #14628:
URL: https://github.com/apache/superset/pull/14628#issuecomment-1041311520


   Going through old PRs and noticed this one. To continue this discussion, I like the idea of moving the export buttons to a more general export dropdown. However, to reiterate my original concern, I still feel adding XML export not really adding any new functionality, as it's really just exporting tabular data but in XML format (=same as the JSON export but in a nowadays less popular format). In addition, it will add maintenance burden to Superset developers, and is IMO a distraction from the core mission of Superset. If someone really wants to convert the data to XML, there's probably great tools for doing that from CSV/JSON formats.
   
   But if others feel otherwise I'm happy to resume this discussion if this is a highly requested feature.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org