Skip to content

Use new logbook streaming websocket api for cases where we need end_time#12753

Merged
balloob merged 3 commits intohome-assistant:devfrom
bdraco:logbook_always_stream
May 23, 2022
Merged

Use new logbook streaming websocket api for cases where we need end_time#12753
balloob merged 3 commits intohome-assistant:devfrom
bdraco:logbook_always_stream

Conversation

@bdraco
Copy link
Member

@bdraco bdraco commented May 23, 2022

Needs home-assistant/core#72351

Proposed change

  • By subscribing we can return results in chunks so
    they are not waiting for a large select on the backend

Last week is fast now demo:
last_week_is_fast_now

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:

- By subscribing we can return results in chunks so
  they are not waiting for a large select on the backend
@balloob balloob merged commit 067c2fd into home-assistant:dev May 23, 2022
@github-actions github-actions bot locked and limited conversation to collaborators May 24, 2022
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