jeroenterheerdt / HAsmartirrigation

Smart Irrigation custom component for Home Assistant
MIT License
301 stars 47 forks source link

Can't add a third zone ... #330

Closed iz3man closed 2 months ago

iz3man commented 2 months ago

What happened?

After updating today to the new release I had to reinstall everything. So far so fine, and I saved the old data. Tried adding new zones. Zone 1 and 2 were already there. Added Zone "Rasen" & "Pool". All fine. Added a third zone "Seite" and the "Rasen" was replaced by "Seite". Added "Hochbeet" and "Pool" was replaced by that. But no new zones were added. So it's limited to two zones.

config_entry-smart_irrigation-5a0782fe66180b6a3eb3d08a58000731.json

How to reproduce

See above

Relevant log output

No response

Which version are you running?

v2024.1.0

Diagnostics file

Additional information

No response

jeroenterheerdt commented 2 months ago

So sorry to hear this. Did you by any chance remove the default zones after you created the first and before you created the second? I am wondering if these types of bugs should prompt me to not have default zones.

Get Outlook for Androidhttps://aka.ms/AAb9ysg


From: iz3man @.> Sent: Wednesday, February 21, 2024 7:34:36 AM To: jeroenterheerdt/HAsmartirrigation @.> Cc: Assign @.>; Subscribed @.> Subject: [jeroenterheerdt/HAsmartirrigation] Can't add a third zone ... (Issue #330)

What happened?

After updating today to the new release I had to reinstall everything. So far so fine, and I saved the old data. Tried adding new zones. Zone 1 and 2 were already there. Added Zone "Rasen" & "Pool". All fine. Added a third zone "Seite" and the "Rasen" was replaced by "Seite". Added "Hochbeet" and "Pool" was replaced by that. But no new zones were added. So it's limited to two zones.

config_entry-smart_irrigation-5a0782fe66180b6a3eb3d08a58000731.jsonhttps://github.com/jeroenterheerdt/HAsmartirrigation/files/14361588/config_entry-smart_irrigation-5a0782fe66180b6a3eb3d08a58000731.json

How to reproduce

See above

Relevant log output

No response

Which version are you running?

v2024.1.0

Diagnostics file

Additional information

No response

— Reply to this email directly, view it on GitHubhttps://github.com/jeroenterheerdt/HAsmartirrigation/issues/330 or unsubscribehttps://github.com/notifications/unsubscribe-auth/AB6PIPQ6O76OIR3IFOZXR43YUYHYZBFKMF2HI4TJMJ2XIZLTSSBKK5TBNR2WLJDUOJ2WLJDOMFWWLO3UNBZGKYLEL5YGC4TUNFRWS4DBNZ2F6YLDORUXM2LUPGBKK5TBNR2WLJLJONZXKZNENZQW2ZNLORUHEZLBMRPXI6LQMWBKK5TBNR2WLKRSGA2TGOBZGQYTON5ENZQW2ZNJNBQXGX3MMFRGK3ECUV3GC3DVMWSHI4TVMWSG4YLNMWXHOYLUMNUF6YLDORUXM2LUPGWHG5LCNJSWG5C7OR4XAZNFJFZXG5LFUZ2G64DJMNZZHAVEOR4XAZNKOJSXA33TNF2G64TZUV3GC3DVMWUTENRTGE3TSMJXG2BKI5DZOBS2K2LTON2WLJLWMFWHKZNKGIYTINZQGU3DQOJUQKSHI6LQMWSWYYLCMVWKK5TBNR2WLKRSGA2TGOBZGQYTON5HORZGSZ3HMVZKMY3SMVQXIZI. You are receiving this email because you were assigned.

Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

iz3man commented 2 months ago

No big deal. I'm pretty sure you'll get this fixed in no time :) And I started updating in "no irrigation season", so there's lot of time to get that fixed.

When I installed the integation I had TWO default zones. Can remember what they were called. Default [12] maybe? Btw: I was already wondering where they came from, as I didn't add them. So imho having "default" zones is not necessary and may add confusion. Because someone may ask: Why? Why two? And then starts renaming or removing them which may add more issues. Just my 2c.

And something more to add: Really happy that you did this great overhaul. Much appreciated. They setup and useage is so much more transparent that way and much faster (well as soon as I can add my additional zones at least ;) ).

sumnerboy12 commented 2 months ago

Hi - just to note that I am seeing the same behaviour when trying to add a third zone. I deleted the original 2 default zones that were created after install.

iz3man commented 2 months ago

@jeroenterheerdt Any update on that issue? There's still month until irrigation season starts though :)

jeroenterheerdt commented 2 months ago

try this: https://github.com/jeroenterheerdt/HAsmartirrigation/releases/tag/v2024.3.0 - that doesn't create default zones.

iz3man commented 2 months ago

Wow. Great. Works now! Though I had to remove the integration and do a home assistante restart. Just removing the integration and updating it didn't work. Case can be closed. Thank you!