Taragolis commented on PR #26374: URL: https://github.com/apache/airflow/pull/26374#issuecomment-1245952818
> Can you explain why? 1. `SqlToSlackOperator` intends to use Slack Webhook ether based on Slack App or Slack Legacy Integrations but both won't support send file, which only allowed by Slack API. Slack Incoming Webhooks and Slack API are not not interchangeable and do not have common interface. 2. Expected different parameters, so if extend current operator than required 4-5 additional parameters which won't use in case of use send as a regular message. 3. I thought also easier maintain something simple rather than with complicated logic and a lot of additional conditions -- 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]
