mistercrunch commented on issue #30584: URL: https://github.com/apache/superset/issues/30584#issuecomment-2448239430
The comment above would take care of making sure the API request returns a json payload as expected, but unclear if the frontend would handle that well out of the box or whether it needs to be touched. Next steps for someone here would be to: - craft a bad zip file that should trigger an error - add the @handle_api_exception decorator to the specific/related endpoint(s) - see how the frontend handles it bonus points for: - surfacing a more useful exception than "import has failed" by bubbling up the right user-relevant error message safely -- 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]
