search

MTrab / landroid_cloud

Landroid Cloud component for Home Assistant
GNU General Public License v3.0
272 stars 33 forks source link

Second Landroid missing after upgrade #146

Closed spiralfive closed 2 years ago

spiralfive commented 2 years ago

I upgraded to the 2.x version yesterday of the integration and noticed today that only one of my Landroids (of two) is present. I tried setting up the integration a second time, but that did not work as it says my account is already present. I also tried deleting the integration and re-adding it, but again only one was added (the same one).

I do not see any errors in the system log related to this integration, if there is a log file elsewhere, I can look at/share I will check there. I can control both Landroids in the app without any issue, and prior to upgrading could using the integration as well.

image

image

MTrab commented 2 years ago

I have not had the chance to test with multiple devices on version 2, so I took a chance.

It would be very helpful, if I could lend access to your Worx account for fixing this issue. You can share your credentials on github_2022 (at) trab.dk

spiralfive commented 2 years ago

I can do that - just send my credentials to that email?

Get Outlook for iOShttps://aka.ms/o0ukef

From: Malene Trab @.> Sent: Thursday, June 2, 2022 10:52:08 AM To: MTrab/landroid_cloud @.> Cc: spiralfive @.>; Author @.> Subject: Re: [MTrab/landroid_cloud] Second Landroid missing after upgrade (Issue #146)

I have not had the chance to test with multiple devices on version 2, so I took a chance.

It would be very helpful, if I could lend access to your Worx account for fixing this issue. You can share your credentials on github_2022 (at) trab.dk

— Reply to this email directly, view it on GitHubhttps://github.com/MTrab/landroid_cloud/issues/146#issuecomment-1144958557, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AGHDAG2BTEGWLIZXBO3BGPLVNDDBRANCNFSM5XVGRJ6A. You are receiving this because you authored the thread.Message ID: @.***>

MTrab commented 2 years ago

Yes, that would be great

spiralfive commented 2 years ago

Alright, I sent it. Let me know before you test the second one if you can - I need to place it outside a fenced area so it can run. Right now it is inside trapped with sprinklers/hoses it could run over.

On Thu, Jun 2, 2022 at 11:18 AM Malene Trab @.***> wrote:

Yes, that would be great

— Reply to this email directly, view it on GitHub https://github.com/MTrab/landroid_cloud/issues/146#issuecomment-1144988023, or unsubscribe https://github.com/notifications/unsubscribe-auth/AGHDAG3IHOYAAYJZVFDRDEDVNDGEZANCNFSM5XVGRJ6A . You are receiving this because you authored the thread.Message ID: @.***>

MTrab commented 2 years ago

Don't think I need to send commands to it - if it loads it'll work ;)

jcustenborder commented 2 years ago

If you need another with multiple I can help as well. I have 5. 2x WR150s 2x WR155s. 1x that was stolen that is just sitting in the account. They all are in random states at the moment as well. One with a broken wire that I'm gonna fix soon.

MTrab commented 2 years ago

@jcustenborder could be great - think I have it working in my development, but all devices are in docked state right now. So just for the sake of testing before release, you are more than welcome to send credentials on github_2022 (at) trab.dk

arcano81 commented 2 years ago

Same issue also here. I have 2 landroids WR130e and WR167e but only one is showing in the app. Both robots are at home mowing the garden!! @MTrab Let me know if you need access to account for testing.

jcustenborder commented 2 years ago

@MTrab I just sent you the credentials! Thanks for taking a look! Right now I have via the app.

I'm still working through all the obstacles in the yard and placements of radio links.

MTrab commented 2 years ago

Thanks - I'm away for the weekend, but will test when I get back home

MTrab commented 2 years ago

I have 2 landroids WR130e and WR167e but only one is showing in the app.

@arcano81 your devices are newer than the other I have test access to right now - could be nice to confirm checks and functions of newer devices is working as well, before releasing the next version. Please send me the credentials at github_2022 (at) trab.dk if you are still willing to help out 🙂

lucrom61 commented 2 years ago

Hello. I have three Landroids. WR104SI WR102SI.1 WR142E

After the update I only see the WR142E

lazyjonez commented 2 years ago

Don't think I need to send commands to it - if it loads it'll work ;)

i got the same error as Spiralfive and also the Integration was renamed in the UI to config yaml as you can see in one of the photos.

Thanks for your efforts and let us know how it goes.

D75DB710-9C3B-4639-83A4-F06113A55623 09FEA740-7DA4-453A-9209-7E6F53995263

MTrab commented 2 years ago

@lazyjonez the reason for the rename is because it was imported from your configuration.yaml, so that is as expected.

As for the missing device - that's the same for everyone. That part I have fixed in my dev-environment, but I'm having a battle with mapping the services correct, so rewriting the whole service-add-function from scratch, to comply with the current guidelines.

MTrab commented 2 years ago

FINALLY think I got it sorted with matching service calls to the correct device 😄 Now I just need to test it thoroughly and fix the last small known bug 🙂

HKUser1 commented 2 years ago

Ok as i understand that right I only have to be patiened and if you has fixed the last bugs,my second Landroid will work too. If i can do something to help you, let me know. I Have one Landroid 2019 and one 2021 Size L

MTrab commented 2 years ago

Correct - it will be fixed in the next release. Have had a sh*t load of bugs with mapping the services correct. But I'm getting close to the target now.

MTrab commented 2 years ago

Just a small update, so you don't think nothing is happening. Got the discovery of multiple mowers in an account sorted 🎊 Then I noticed some strange thing happen in the mapping from service <-> device, forcing me to rewrite the whole service registration section. Now I see some excessive spamming of the MQTT endpoint resulting in temporary bans, which I'm currently tracking down.

A version 2.1.0 will be released as soon as I got this MQTT issue sorted.

Sorry about the long wait - my day-job takes up way too much of my sparetime for being a neard and coding HA integrations 😁

MTrab commented 2 years ago

FINALLY everything seems sorted. Unfortunately the new release, which is coming up shortly, WILL bring new breaking changes. Had to rewrite huge chunks of the code, to fix these issues and keep compatibility between the different brands.

@Barma-lej need to do some work again (sorry), for the HALandroid package to function with version 2.1.0

arcano81 commented 2 years ago

Credentials sent to your email! Waiting for bug fixes 😀😀

Thankyou!

Alessandro Rondelli

Il giorno mar 7 giu 2022 alle ore 09:31 Malene Trab < @.***> ha scritto:

I have 2 landroids WR130e and WR167e but only one is showing in the app.

Your devices are newer than the other I have test access to right now - could be nice to confirm checks and functions of newer devices is working as well, before releasing the next version. Please send me the credentials at github_2022 (at) trab.dk if you are still willing to help out 🙂

— Reply to this email directly, view it on GitHub https://github.com/MTrab/landroid_cloud/issues/146#issuecomment-1148301641, or unsubscribe https://github.com/notifications/unsubscribe-auth/ASG2GNP3SZETXT47M75CMS3VN33EXANCNFSM5XVGRJ6A . You are receiving this because you commented.Message ID: @.***>

MTrab commented 2 years ago

This is an old, closed issue. This have been resolved months ago.

Other than that, nothing received.