search

dan-r / HomeAssistant-NissanConnect

Unofficial Home Assistant integration for interacting with Nissan Connect vehicles 🚗
MIT License
45 stars 12 forks source link

Two cars #43

Closed chemicalandy closed 2 weeks ago

chemicalandy commented 2 months ago

I've been using this integration for a few months and it's been working well. Last week I acquired a second LEAF and have added this to the integration as a second entry (the new car is using a different Nissan account to the old one). This creates a new device for the second car but with no entities. After leaving both cars set up for a day, the entities moved from the old car to the new one, so at first I could only see the old car and now I can only see the new one.

Is there any way of using the integration to see both cars at the same time. (It's possible to add two cars to the same Nissan account in the Nissan app. When I did this the second car wasn't found by the integration - also I'd prefer them to be in separate accounts).

dan-r commented 2 months ago

As you see from the weird behaviour it should work but I've never tested it. Would you mind emailing me (address on my GitHub profile), and I can work on it with you directly.

dan-r commented 3 weeks ago

This should be fixed under v0.6.0-beta.1. Would you mind trying it out?

I'd recommend removing all NissanConnect accounts first, then re-configuring them.

chemicalandy commented 3 weeks ago

I've tried the new version and it doesn't seem any different to the previous version.

I assume I've got the new version installed correctly - I copied the files for version 6 into the nissan_connect folder in custom_components, restarted Home Assistant, deleted the existing account and then added two accounts (one for each car). The first account works as expected, but the second has no devices or entities. However, I can see from my history that it briefly got the battery data from the second car before going back to the first car.

Regards

Andy

On Wed, 30 Oct 2024 at 11:37, Dan Raper @.***> wrote:

This should be fixed under v0.6.0-beta.1. Would you mind trying it out?

I'd recommend removing all NissanConnect accounts first, then re-configuring them.

— Reply to this email directly, view it on GitHub https://github.com/dan-r/HomeAssistant-NissanConnect/issues/43#issuecomment-2446760026, or unsubscribe https://github.com/notifications/unsubscribe-auth/A6RAHY43V3KDNFKAZ4QUK53Z6DAF7AVCNFSM6AAAAABOAN7KVKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINBWG43DAMBSGY . You are receiving this because you authored the thread.Message ID: @.***>

cameronmarklewis commented 2 weeks ago

v0.6.0-beta.1 works with two cars for me, thanks! I removed my NissanConnect account before restarting Home Assistant and followed the UI to add again without issue.

dan-r commented 2 weeks ago

@cameronmarklewis glad it’s working for you! Just to confirm, is that two cars in one account or two separate accounts?

dan-r commented 2 weeks ago

@chemicalandy I’d recommend using HACS to manage updates. Within HACS, you just need to find the integration from the HACS UI, click re-download and choose the beta. If you’re doing it manually:

  1. Delete the nissan_connect folder from custom_components in your installation. There’s no configuration stored in here, so it’s safe to do so.
  2. Download the beta version from here
  3. Copy the custom_components/nissan_connect folder from the zip file to the custom_components folder in home assistant

I’ll take another look at the code tomorrow to see if I can spot anything else that might be causing an issue.

cameronmarklewis commented 2 weeks ago

@cameronmarklewis glad it’s working for you! Just to confirm, is that two cars in one account or two separate accounts?

Two cars in one account, in the previous version only one car loaded. All seems to be working too, heating/ flash lights etc happens on the correct car.

chemicalandy commented 2 weeks ago

@dan-r I've installed the beta version using HACS and added the second car (it's in a separate account). It's all working now! Thanks for sorting this out.

On Tue, 5 Nov 2024 at 23:23, Dan Raper @.***> wrote:

@chemicalandy https://github.com/chemicalandy I’d recommend using HACS to manage updates, but if you’re doing it manually:

  1. Delete the nissan_connect folder from custom_components in your installation. There’s no configuration stored in here, so it’s safe to do so.
  2. Download the beta version from here https://github.com/dan-r/HomeAssistant-NissanConnect/releases/tag/v0.6.0-beta.1
  3. Copy the custom_components/nissan_connect folder from the zip file to the custom_components folder in home assistant

I’ll take another look at the code tomorrow to see if I can spot anything else that might be causing an issue.

— Reply to this email directly, view it on GitHub https://github.com/dan-r/HomeAssistant-NissanConnect/issues/43#issuecomment-2458395592, or unsubscribe https://github.com/notifications/unsubscribe-auth/A6RAHYZCQFA5AL6FSXHQ43TZ7FHOZAVCNFSM6AAAAABOAN7KVKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINJYGM4TKNJZGI . You are receiving this because you were mentioned.Message ID: @.***>

dan-r commented 2 weeks ago

Great news @chemicalandy. I'll put that out in a proper release some time this week (it's been a hectic one!). Could you and @cameronmarklewis please let me know if you have any issues with the multi car support (multiple accounts and multiple cars on one account respectively), as this isn't something I can test.

Dan

dan-r commented 2 weeks ago

Resolved by #70