Skip to content

fix: skip CMD_DOORBELL_SET_PAYLOAD for E340 on MiniBase Chime#834

Open
lenoxys wants to merge 1 commit intobropat:developfrom
lenoxys:fix/e340-minibase-chime-livestream
Open

fix: skip CMD_DOORBELL_SET_PAYLOAD for E340 on MiniBase Chime#834
lenoxys wants to merge 1 commit intobropat:developfrom
lenoxys:fix/e340-minibase-chime-livestream

Conversation

@lenoxys
Copy link
Copy Markdown
Contributor

@lenoxys lenoxys commented Mar 4, 2026

Summary

  • The E340 doorbell was unconditionally routed to CMD_DOORBELL_SET_PAYLOAD (1700) for livestreaming, but the MiniBase Chime station rejects this command with ERROR_INVALID_PARAM (-110)
  • When the station is a MiniBase Chime, fall through to the default CMD_SET_PAYLOAD (1350) path which works correctly
  • Adds isStationMiniBaseChime() helper methods (static, by-serial, and instance) for reuse

Test plan

  • E340 behind MiniBase Chime: livestream should use CMD_SET_PAYLOAD and succeed
  • E340 behind HomeBase 3: livestream should still use CMD_DOORBELL_SET_PAYLOAD

Resolves homebridge-plugins/homebridge-eufy-security#877

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Video Doorbell E340 with mini home base chime not streaming video using x64 system.

1 participant