Skip to content

Make error optional in connection lost service check#8937

Merged
bramkragten merged 1 commit intohome-assistant:devfrom
bdraco:stop_restart_tweaks
Apr 20, 2021
Merged

Make error optional in connection lost service check#8937
bramkragten merged 1 commit intohome-assistant:devfrom
bdraco:stop_restart_tweaks

Conversation

@bdraco
Copy link
Copy Markdown
Member

@bdraco bdraco commented Apr 17, 2021

Proposed change

Make error optional in connection lost service check

Type of change

  • Dependency upgrade
  • Bugfix (non-breaking change which fixes an issue)
  • New feature (thank you!)
  • Breaking change (fix/feature causing existing functionality to break)
  • Code quality improvements to existing code or addition of tests

Example configuration

Additional information

  • This PR fixes or closes issue: fixes #
  • This PR is related to issue or discussion:
  • Link to documentation pull request:

Checklist

  • The code change is tested and works locally.
  • There is no commented out code in this PR.
  • Tests have been added to verify that the new code works.

If user exposed functionality or configuration variables are added/changed:

Comment thread src/state/connection-mixin.ts Outdated
Comment thread src/state/connection-mixin.ts Outdated
@bramkragten
Copy link
Copy Markdown
Member

Oh shit, I now see we already threw the error before, after showing the toast... so this PR is not needed 🙈

Sorry! My bad... 🤦

The comment by @spacegaier should be addressed btw...

@bdraco bdraco force-pushed the stop_restart_tweaks branch from e517f4c to 6389c82 Compare April 20, 2021 00:40
@bdraco bdraco changed the title Return undefined when on connection lost due to stop/restart Make error optional in connection lost service check Apr 20, 2021
@bramkragten bramkragten merged commit f21ed24 into home-assistant:dev Apr 20, 2021
@github-actions github-actions Bot locked and limited conversation to collaborators Apr 21, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants