DGNum/uptime-kuma-api
6
0
Fork 0
Code Issues 17 Pull requests 5 Projects Releases 24 Packages Wiki Activity Actions

fix: check only for required notification arguments

Browse source
This commit is contained in:
lucasheld 2023-05-20 22:06:05 +02:00
parent ce1cc12740
commit 8a0ad53753
2 changed files with 4 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
tests/test_notification.py
View file

@ -13,8 +13,9 @@ class TestNotification(UptimeKumaTestCase):
"name": "notification 1", "name": "notification 1",
"isDefault": True, "isDefault": True,
"applyExisting": True, "applyExisting": True,
"type": NotificationType.PUSHBYTECHULUS, "type": NotificationType.TELEGRAM,
"pushAPIKey": "123456789" "telegramChatID": "123456789",
"telegramBotToken": "987654321"
} }
# test notification # test notification

2
uptime_kuma_api/api.py
View file

@ -305,7 +305,7 @@ def _check_arguments_notification(kwargs) -> None:
_check_missing_arguments(required_args, kwargs) _check_missing_arguments(required_args, kwargs)
type_ = kwargs["type"] type_ = kwargs["type"]
required_args = notification_provider_options[type_] required_args = [i for i, j in notification_provider_options[type_].items() if j["required"]]
_check_missing_arguments(required_args, kwargs) _check_missing_arguments(required_args, kwargs)
_check_argument_conditions(notification_provider_conditions, kwargs) _check_argument_conditions(notification_provider_conditions, kwargs)