Skip to content

Fix remote buzzer reconnect handling#1582

Open
reloxx13 wants to merge 2 commits into
PhotoboothProject:devfrom
reloxx13:fix/remotebuzzer-reconnect
Open

Fix remote buzzer reconnect handling#1582
reloxx13 wants to merge 2 commits into
PhotoboothProject:devfrom
reloxx13:fix/remotebuzzer-reconnect

Conversation

@reloxx13

@reloxx13 reloxx13 commented Jun 22, 2026

Copy link
Copy Markdown
Collaborator

Prerequisites checklist

What is the purpose of this pull request? (put an "x" next to an item)

  • Documentation update
  • Bug fix
  • New feature
  • Other, please explain:

What changes did you make? (Give an overview)

  • remove the one-shot remote buzzer reachability probe in the browser client
  • connect to the remote buzzer server directly via socket.io with reconnect enabled
  • track connect, disconnect, and connect_error so the client state reflects the real connection

Is there anything you'd like reviewers to focus on?

  • whether reconnect and disconnected-state handling behave correctly when the remote buzzer server is unavailable during page load and comes back later

AI used to create this Pull Request?

  • AI was used to help implement the change.
  • The resulting code change is limited to assets/js/remotebuzzer-client.js and updates only the client connection/reconnect behavior.
  • Validation run for this change:
    • npm run eslint -- assets/js/remotebuzzer-client.js
    • npm run build:gulp

@reloxx13 reloxx13 marked this pull request as ready for review June 22, 2026 08:52
@reloxx13 reloxx13 marked this pull request as draft June 22, 2026 09:35
@reloxx13 reloxx13 marked this pull request as ready for review June 22, 2026 19:06
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.

1 participant