Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feat: telegram send file Operator and Hook #44040

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

HaKkaz
Copy link
Contributor

@HaKkaz HaKkaz commented Nov 14, 2024

I found that the Slack provider has a send_file hook and related operator, so I want to implement a similar one for the Telegram provider!

If this is a good idea, I will add unit tests and any other necessary components later!

My first PR, which was related to unit tests, was submitted last month, so I am still a newcomer. Any suggestions or feedback on this work would be greatly appreciated!


^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in newsfragments.

- correct the key name in the kwargs to match the argument name of send_file
- replace send_message with send_file
@potiuk
Copy link
Member

potiuk commented Nov 14, 2024

sure. Go ahead.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants