release-argus / Argus

Argus is a lightweight monitor to notify of new software releases via Gotify/Slack/other messages and/or WebHooks.
https://release-argus.io
Apache License 2.0
296 stars 13 forks source link

bug: Choosing Notify global dropdown gives blank screen when it does not match Type #449

Closed BaccanoMob closed 1 month ago

BaccanoMob commented 1 month ago

I have a notify config for ntfy in the config.yml file (for example, say ntfy_global).

When I use the WebUI to create a service and then add notify, the notify form pops up. When I select ntfy_global in the dropdown, the screen goes blank.

Workaround: Choose the type drop down field first to ntfy then choose the global.

Solution: Choosing global should first modify the Type drop down field (possibly disable this field while at it) before setting the global settings.

JosephKav commented 1 month ago

Thanks for reporting this :). Took a look and it seemed to be where it checks whether the .actions are for a different notify type. But if the currently selected type didn't have a .actions, it would try and loop over undefined and get a TypeError. Fixed this by just defaulting that var to an empty list.