Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Integration BMW Connected Drive - Change account / Re-Authorize #86253

Closed
dyarfaradj opened this issue Jan 19, 2023 · 7 comments · Fixed by #90251
Closed

Integration BMW Connected Drive - Change account / Re-Authorize #86253

dyarfaradj opened this issue Jan 19, 2023 · 7 comments · Fixed by #90251

Comments

@dyarfaradj
Copy link

The problem

As the title says. It would be really helpful to let users change account or reauthorize. I've changed password of my BMW Connected Drive account and now none of the integration works. I don't want to remove it and install it again, I will loose all the scripts it's attached to etc.

What version of Home Assistant Core has the issue?

2022.12.1

What was the last working version of Home Assistant Core?

2022.12.1

What type of installation are you running?

Home Assistant OS

Integration causing the issue

BMW Connected Drive

Link to integration documentation on our website

https://www.home-assistant.io/integrations/bmw_connected_drive

Diagnostics information

No response

Example YAML snippet

No response

Anything in the logs that might be useful for us?

No response

Additional information

No response

@home-assistant
Copy link

Hey there @gerard33, @rikroe, mind taking a look at this issue as it has been labeled with an integration (bmw_connected_drive) you are listed as a code owner for? Thanks!

Code owner commands

Code owners of bmw_connected_drive can trigger bot actions by commenting:

  • @home-assistant close Closes the issue.
  • @home-assistant rename Awesome new title Change the title of the issue.
  • @home-assistant reopen Reopen the issue.
  • @home-assistant unassign bmw_connected_drive Removes the current integration label and assignees on the issue, add the integration domain after the command.

(message by CodeOwnersMention)


bmw_connected_drive documentation
bmw_connected_drive source
(message by IssueLinks)

@rikroe
Copy link
Contributor

rikroe commented Jan 19, 2023

To be honest, not sure if this is even support by HA... Maybe some of the devs can answer this.

As a workaround, you could edit config/.storage/core.config_entries, search for bmw_connected_drive and update your password there. Make sure to take a backup of that file before and that it is still a valid JSON after your edit.
Then restart HA and the integration should work again.

@cdnninja
Copy link
Contributor

Re-auth flow is supported by HA but has to be implemented by the integration. https://developers.home-assistant.io/docs/config_entries_config_flow_handler/#reauthentication this would need to be added to this integration.

@dyarfaradj
Copy link
Author

To be honest, not sure if this is even support by HA... Maybe some of the devs can answer this.

As a workaround, you could edit config/.storage/core.config_entries, search for bmw_connected_drive and update your password there. Make sure to take a backup of that file before and that it is still a valid JSON after your edit. Then restart HA and the integration should work again.

Yeah, this did the trick. Thank you.
But would be nice to see if "Re-Auth" can be easily integrated like @cdnninja mentioned

@manuelcarvajal
Copy link

Les funciona BMW connected? llevo meses que me ha dejado de funcionar

@rikroe
Copy link
Contributor

rikroe commented Jan 28, 2023

Solved with work around. Implementing the reauth seems to require extensive nchanges to the library as well which I don't have time for right now.

@rikroe
Copy link
Contributor

rikroe commented Jan 28, 2023

@home-assistant close

@github-actions github-actions bot locked and limited conversation to collaborators Feb 27, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants