justin-tomlinson commented on PR #20261:
URL: https://github.com/apache/superset/pull/20261#issuecomment-1155943776

   Just checking if I understand this pull request correctly. Currently the CSV 
download option is not available in 1.5.0 when embedding using the sdk. I 
pulled master and enabled embedding and embedded in a test page and was able to 
see the option to download the csv from a chart, but selecting the download 
raised errors in the console about the iframe not having certain permissions 
(allow-forms, allow popups). Does this pull request correct issue allow user of 
an embedded dashboard to download the csv from charts?


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to