Skip to content

Build 933: Panel offline reconnect fix#5

Merged
mkalhitti-cloud merged 1 commit into
mainfrom
build/933-panel-offline-reconnect
Mar 1, 2026
Merged

Build 933: Panel offline reconnect fix#5
mkalhitti-cloud merged 1 commit into
mainfrom
build/933-panel-offline-reconnect

Conversation

@mkalhitti-cloud
Copy link
Copy Markdown
Owner

Problem

The V12_001 Panel hard-fails silently when it can't connect to the Strategy (market closed / Strategy not yet in State.Realtime). The LED turns grey and stays grey — no retry kicks in.

Root Cause

The Panel already has ScheduleReconnect() which retries every 3 seconds, but this was only called after a live connection dropped. The initial ConnectToStrategy() catch block did not call it, so the first failure was terminal until the user manually interacted.

Fix (Panel-side only, 1 line)

  • Added ScheduleReconnect() to ConnectToStrategy() catch block
  • Updated LED tooltip: 'Waiting for Strategy (retrying...)' so user knows it is actively retrying
  • No Strategy-side changes

Files Changed

  • src/V12_001.cs only (+5 lines, -1 line)

Expected Behavior After Fix

  1. Panel loads → tries to connect → fails (Strategy offline)
  2. LED shows grey with tooltip Waiting for Strategy (retrying...)
  3. NT8 log shows V12.14: Auto-reconnect attempting... every 3 seconds
  4. Strategy comes online → LED turns green automatically

- Add ScheduleReconnect() to ConnectToStrategy() catch block
  so the retry loop starts immediately on initial connection failure
  (market closed / Strategy not yet in Realtime state)
- Update LED tooltip: 'Waiting for Strategy (retrying...)' so
  user can distinguish active-retry from permanent-disconnect
- No Strategy-side changes  Panel-only fix
@mkalhitti-cloud mkalhitti-cloud merged commit 8273a30 into main Mar 1, 2026
@mkalhitti-cloud mkalhitti-cloud deleted the build/933-panel-offline-reconnect branch March 4, 2026 19:43
mkalhitti-cloud added a commit that referenced this pull request May 20, 2026
…connect

Build 933: Panel offline reconnect fix
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