john-bodley commented on pull request #16077: URL: https://github.com/apache/superset/pull/16077#issuecomment-899624313
@betodealmeida I don't have much context on the query context change, but when I read, the following, > To get the data for a given chart, we need to POST the query_context to /api/v1/chart/ i.e., a `GET` request first requires a `POST` (to I gather the same table) I just wanted to check that we're confident in the pattern that's being used. Per _this_ PR it feels like _that_ change is snowballing and resulted in unforeseen behavior and thus thought there was merit in confirming that collectively the community was onboard with this approach. -- 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]
