Skip to content

[Bug] Disabled notification alerts are reported as diagnostic failures #29

Closed

Description

@stephenjharrington

ChannelWatch version

1.6.1

Channels DVR version

v2026.10.24.2341

Host OS / platform

Synology DSM 7.4.1-90080 Update 1

Docker version

Docker 24.0.2-1706 / Synology Container Manager

Number of Channels DVR servers monitored

1

docker-compose.yml snippet

Steps to reproduce

  • Start ChannelWatch
  • Got to Dianostics | Diagnostics Tests
  • Hit "Run All Tests" Button
  • All tests fail, even thought it all seems to be running perfectly? (See Screenshot)

Expected behavior

All tests should run and pass.

Actual behavior

All tests show an Error.

Logs


Additional context

Not sure if I'm doing something wrong or there is some other pre-requisite for running the Diagnostics but it's not working for me!

:-)

Image

Activity

  1. CoderLuii commented on Oct 26, 2026

    @CoderLuii
    Owner

    @stephenjharrington, fixed in v1.6.2. The diagnostic buttons were sending their display labels instead of the API commands, so every test was rejected before it ran. Individual tests and Run All Tests now call the right checks.

    Install it from Settings > Updates without recreating your container. Notification tests still send sample alerts through configured destinations, and delivery can fail if a destination isn't configured or reachable.

    v1.6.2 release notes

  2. stephenjharrington commented on Oct 27, 2026

    @stephenjharrington
    Author

    Thanks @CoderLuii , updated to 1.6.2 and now only 3 out of 10 tests "fail".

    Upon further examination it looks like it's because I have those 3 items diabled in Alerts.
    I was on "Balanced" Alerts. If I change to "Everything" Preset then all the tests pass!

    Perhaps for Alerts that are disabled, the corresponding Diagnostics test should either be greyed out and/or disabled, rather than saying it failed, which is a bit misleading?

    Image Image
  3. changed the title [-][Bug]:[/-] [+][Bug] Disabled notification alerts are reported as diagnostic failures[/+] on Oct 27, 2026
  4. CoderLuii commented on Oct 27, 2026

    @CoderLuii
    Owner

    @stephenjharrington, fixed in v1.6.3. Disabled notification tests now show Skipped, with a reason and a separate count. They do not send a sample. You can keep Balanced selected; its three disabled alerts no longer count as failures.

    Tests respect the checked DVR's effective settings, including per-DVR overrides. Tested DVR shows which server was checked. Genuine connection and delivery errors still show as failures.

    Install from Settings > Updates without recreating your container. Thanks for catching the misleading results.

    v1.6.3 release notes

  5. CoderLuii commented on Oct 27, 2026

    @CoderLuii
    Owner

    @stephenjharrington, here's the before/after comparison with demo data: 7 passed, 3 failed becomes 7 passed, 0 failed, 3 skipped. The three disabled alerts stay disabled and are no longer counted as failures.

    Before and after: disabled notification alerts show Skipped instead of Failed, with separate result counts

  6. stephenjharrington commented on Oct 27, 2026

    @stephenjharrington
    Author

    Have updated to 1.6.3 and tested/verified fixes.
    Great work @CoderLuii , thanks again!

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

Metadata

Metadata

Assignees

Labels

type/bugSomething isn't working

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions