home-assistant / alerts.home-assistant.io

Home Assistant Alerts
https://alerts.home-assistant.io
37 stars 71 forks source link

Remove outdated UniFi Protect alert #639

Closed TheJulianJES closed 2 months ago

TheJulianJES commented 2 months ago

Proposed change

This removes the old alert about UniFi Protect 3.x being incompatible, as it's compatible with HA since multiple months now.

Additional information

In the mean time, there were issues with UniFi Protect 4.x and UniFi OS 4.x which were addressed in 2024.6.2 and 2024.6.3, with further improvements coming in 2024.7.0. As of now, there are no known compatibility issues.

As pointed out internally, this alert technically doesn't follow guidelines, as it creates a repair that's shown to users, even if their Protect integration is working correctly (if they are running an older Protect and HA version). That's also why the alert doesn't mention UniFi OS 4.x or UniFi Protect 4.x:

Since the alert also creates a persistent banner on the Protect docs page that links to the outdated alert, we should remove it IMO.

Alternative approach

If we do not want to remove the alert completely, another approach would be to change the text to mention Protect/UOS 4.x. This would essentially be like the linked PR, with the exception of limiting the Home Assistant versions impacted to 2024.6.1 or older. After a few months, it could then also be removed completely.

Like with the current version of the alert, it would still be shown to everyone on older HA versions, even if they're not impacted. So, I'm not sure if we're allowed to do that.