donavanbecker / homebridge-rainbird

The Homebridge RainBird plugin allows you to access your RainBird Controller from HomeKit with Homebridge.
ISC License
20 stars 0 forks source link

Bug: Device Removed After Upgrade #538

Closed equals42 closed 2 days ago

equals42 commented 6 days ago

Describe The Bug

Device deleted on upgrade and password section missing

To Reproduce

Create a new device with IP

Expected behavior

I expect to be able to create a device and not see errors about a missing password

Relevant log output

Initial logs from after upgrade of plugin

[11/09/2024, 10:32:39] [RainBird] Verify Config, Error Message: The devices config section is missing from the config. This device will be skipped., Submit Bugs Here: https://bit.ly/homebridge-rainbird-bug-report
[11/09/2024, 10:32:39] [RainBird] Loading accessory from cache: 7
Setup Payload:
X-HM://0024WZTN83F00
Enter this code with your HomeKit app on your iOS device to pair with Homebridge:

    ┌────────────┐     
    │ XXX-XX-XXX │     
    └────────────┘     

[11/09/2024, 10:32:39] [RainBird] Failed to Discover Devices, "this.config.devices is not iterable"

After config of new device in plugin:

[11/09/2024, 10:38:09] [RainBird] Initializing RainBird platform...
[11/09/2024, 10:38:11] [RainBird] Verify Config, Error Message: The devices config section is missing the "Password" in the config, and will be skipped., Submit Bugs Here: https://bit.ly/homebridge-rainbird-bug-report
[11/09/2024, 10:38:11] [RainBird] Loading accessory from cache: 7
Setup Payload:
X-HM://0024WZTN83F00
Enter this code with your HomeKit app on your iOS device to pair with Homebridge:

    ┌────────────┐     
    │ XXX-XX-XXX │     
    └────────────┘     

[11/09/2024, 10:38:11] [RainBird] Failed to Discover Devices, "Cannot read properties of undefined (reading 'refreshRate')"

Config for homebridge-rainbird

{
    "name": "RainBird",
    "devices": [
        {
            "ipaddress": "10.0.X.XX",
            "showRainSensor": true,
            "showValveSensor": true,
            "showDelayIrrigationSwitch": true,
            "syncTime": false
        }
    ],
    "platform": "RainBird"
}

Screenshots

No response

Device & Model

Rainbird N/A

Node.js Version

v20.9.0

NPM Version

unknown

Homebridge Version

1.8.4

Homebridge Rainbird Plugin Version

v2.1.0

Homebridge Config UI X Plugin Version

v4.58.0

Operating System

Raspian

donavanbecker commented 6 days ago

@equals42 What version did you upgrade from?

equals42 commented 4 days ago

I looked through the logs and there seems to be errors back to July on 2.0.2 so maybe it wasn’t this upgrade. I’ve been out of town most of the summer and didn’t mess with the Rainbird in Homekit so it could have happened on a 1.x to 2.x upgrade. It was functional in spring IIRC. Sorry if this is unhelpful.As of now, I cannot seem to enter a password in the device setup so it is not functional[11/09/2024, 10:38:11] [RainBird] Verify Config, Error Message: The devices config section is missing the "Password" in the config, and will be skipped., Submit Bugs Here: https://bit.ly/homebridge-rainbird-bug-report[11/09/2024, 10:38:11] [RainBird] Loading accessory from cache:

donavanbecker commented 4 days ago

Please respond directly on GitHub as you are filling the comments with extra text.

donavanbecker commented 4 days ago

Can you send a screenshot of what your UI looks like?

equals42 commented 3 days ago

IMG_0129

donavanbecker commented 3 days ago

Not sure how the password field got hidden on the UI. That wouldn't cause the password to be deleted, but v2.1.1 adds it back to the UI.

equals42 commented 2 days ago

The password field is back. Once it was entered, the sprinkler worked again. There was some error about ESP-me having invalid characters but it doesn’t seem to impact its functionality.

Thanks.