-
-
Notifications
You must be signed in to change notification settings - Fork 9
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: send alert image with telegram message, if requested #279
base: main
Are you sure you want to change the base?
Conversation
- Dockerfile-dev: image FROM pyronear/pyro-api:python3.8-alpine3.10
- add boolean field send_message to recipient table - add int field media_id to notification table
Nice of you to open a PR 🙏 |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #279 +/- ##
==========================================
- Coverage 94.54% 94.31% -0.23%
==========================================
Files 63 66 +3
Lines 1594 1690 +96
==========================================
+ Hits 1507 1594 +87
- Misses 87 96 +9
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
src/Dockerfile-dev
Outdated
@@ -1,4 +1,4 @@ | |||
FROM pyroapi:python3.8-alpine3.10 | |||
FROM pyronear/pyro-api:python3.8-alpine3.10 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@blenzi your are changing this in severals PR it should be done once in a PR I guess
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @MateoLostanlen. Now that #281 was merged, this will be part of main
# Conflicts: # poetry.lock # pyproject.toml # src/Dockerfile-dev
…ia_id to notification tables
This PR adds the possibility to send the alert image together with a telegram message, if requested