Clear override text conditionally based on device

[bjorkert]

This pull request resolves issue #403.

Description:

This change introduces a conditional clearing of the override information in the infoManager. Previously, the override data was cleared every time the device status was updated.

This caused an issue for Trio users, where the override text would disappear from the display. This happened because for Trio, override information is fetched from treatments, and the device status does not contain this data. If a device status update occurred after the treatment data was processed, the override information would be incorrectly cleared.

For "Loop" devices, the override information is part of the device status, so clearing it upon refresh is the correct behavior to ensure the display is up-to-date.

Changes:

• In updateDeviceStatusDisplay(jsonDeviceStatus:), the call to infoManager.clearInfoData(types: [.override]) is now wrapped in a conditional check.
• The override data is now only cleared if the selected device is "Loop".

This ensures that the override information is handled correctly for both Loop and Trio users, fixing the glitch where the override text would intermittently disappear for the latter.

[marionbarker]

Code review

The code review looks appropriate

Test

I built this branch (as well as dev branch) on a test phone viewing both a Loop URL and Trio URL (at separate times). I did not notice a difference with the Loop URL between the dev and trio-override builds.

Trio URL

• With dev:
  • disabling an override removed the override from the display completely (instead of changing the end point)
• With trio-override:
  • disabling an override updated the end point to the actual duration