-
-
Notifications
You must be signed in to change notification settings - Fork 12
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: Support for Telegram #63
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
JosephKav
force-pushed
the
33-feat-support-for-telegram
branch
from
May 22, 2022 23:16
39ed994
to
9ee4c36
Compare
also includes support Shoutrrr notifiers - see https://containrrr.dev/shoutrrr/v0.5/services/overview/ was a very large rewrite of the notifiers, involving the removal of my slack/gotify code, replacing that with Shoutrrr. There was also a small change in formatting style that will follow elsewhere soon. Have added automatic conversion to the new formatting, so existing slack (or mattermost) and gotify should stay working fingers crossed!
JosephKav
force-pushed
the
33-feat-support-for-telegram
branch
from
May 22, 2022 23:21
9ee4c36
to
0aada49
Compare
Would have called this `deployed_version` originally if I'd planned support to monitor deployed services. Makes more sense as it is tracking the `deployed_version` either through an API, or by triggering update scripts with WebHook(s) Shouldn't effect existing deployments as I've made it convert `current_version*` to `deployed_version*` automatically
JosephKav
force-pushed
the
33-feat-support-for-telegram
branch
from
May 23, 2022 00:24
1f6e529
to
3406e49
Compare
JosephKav
force-pushed
the
33-feat-support-for-telegram
branch
from
May 23, 2022 04:20
22c253c
to
7e30a89
Compare
also remove `approved_version` once it matches `deployed_version`
JosephKav
force-pushed
the
33-feat-support-for-telegram
branch
from
May 23, 2022 13:04
eff68cf
to
14a341c
Compare
JosephKav
force-pushed
the
33-feat-support-for-telegram
branch
from
May 23, 2022 13:17
14a341c
to
097a0cd
Compare
JosephKav
force-pushed
the
master
branch
2 times, most recently
from
May 23, 2022 17:25
35a2f8f
to
fb109d1
Compare
…support-for-telegram
JosephKav
force-pushed
the
33-feat-support-for-telegram
branch
from
May 23, 2022 17:39
4ae7d1c
to
a1fafe8
Compare
JosephKav
force-pushed
the
33-feat-support-for-telegram
branch
from
May 23, 2022 21:44
9ec8500
to
1d1e8a1
Compare
This was referenced May 24, 2022
JosephKav
force-pushed
the
33-feat-support-for-telegram
branch
2 times, most recently
from
May 25, 2022 11:15
e4dc5ba
to
b87f21e
Compare
- raised an issue for telegram not parsing `chats` param as their docs say that it can be a param containrrr/shoutrrr#249 - removed mentions of slack/gotify in the context of the old notify code
JosephKav
force-pushed
the
33-feat-support-for-telegram
branch
3 times, most recently
from
May 25, 2022 17:15
870b22a
to
e78ced7
Compare
JosephKav
force-pushed
the
33-feat-support-for-telegram
branch
from
May 25, 2022 18:16
e78ced7
to
dc36433
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
also includes support for every Shoutrrr notifier - see https://containrrr.dev/shoutrrr/v0.5/services/overview/
was a very large rewrite of the notifiers, involving the removal of my
slack
/gotify
code, replacing that withShoutrrr
. There was also a small change in the formatting style of this new section that will follow elsewhere soon. Have added automatic conversion to the new formatting, so existingSlack
(orMatterMost
) andGotify
should stay working fingers crossed!(Adding
Shoutrrr
did ruin the-h
text, adding loads of-ginkgo.XXX
options that I want to remove/hide)fixes #33
fixes #5