Add new websocket messages for managing station state #62

neilenns · 2024-05-18T20:08:18Z

Fixes #53

Adds three new incoming websocket messages: kSetStationState, kGetStationState, and kGetStationStates
Adds fournew outgoing websocket messages: kStationStateUpdate, kStationStates, and kVoiceConnectedState
Adds a new RadioHelper class with a method to manage setting radio states
Removes all direct radio state setting code from main.cpp and uses RadioHelper instead
Adds a new IPC message station-state-update
Updates an AFV radio with the values provided in kSetStationState messages
Updates the client to respond to station-state-update to refresh the state of a radio based on the values received

See comments on the PR for additional info.

…kAudio into neilenns/kSetStationStatus

backend/include/sdk.hpp

backend/src/main.cpp

…kAudio into neilenns/kSetStationStatus

backend/src/sdk.cpp

pierr3 · 2024-07-20T21:37:52Z

backend/src/main.cpp


-    return Napi::Boolean::New(info.Env(), true);
+    auto result = RadioHelper::SetRadioState(MainThreadShared::mApiServer.get(), newState);


I'd rather you you don't access the raw ptr behind the shared_ptr, if it's currently a unique_ptr change it to a shared_ptr and pass it by reference, that's the whole point of smart pointers

pierr3 · 2024-07-20T21:38:04Z

backend/src/main.cpp

+    newState.headset = false;
+    newState.xca = false;
+
+    RadioHelper::SetRadioState(MainThreadShared::mApiServer.get(), newState);


Same here, no raw pointers please

pierr3 · 2024-07-20T21:38:28Z

backend/src/main.cpp


-    return Napi::Boolean::New(info.Env(), true);
+    auto result = RadioHelper::SetRadioState(MainThreadShared::mApiServer.get(), newState);


Same here for raw pointers

backend/src/sdk.cpp

src/renderer/src/components/bootstrap.tsx

backend/include/sdk.hpp

backend/src/main.cpp

backend/include/RadioHelper.hpp

neilenns added 24 commits May 16, 2024 18:27

Kinda works but kinda doesn't

5ba2fb4

More copying of code from main.cpp

1230c71

Try notifying the UI

b49ab88

Add refreshing of connected clients

98f3bfc

Merge branch 'neilenns/kSetStationStatus' of github.com:neilenns/Trac…

a6a7622

…kAudio into neilenns/kSetStationStatus

Add allRadios to kFrequencyStateUpdate mesage

1f49465

Event gets sent to Electron when frequencies update

859b441

Confirmed flows through to renderer

4dfa190

Respond to updates

91e0c05

Add kStationStateUpdated message

349a4d9

Add the beginnings of kGetStationStates

ec24540

Add kStationStates response

b307ddb

Send message when station states are changed via websocket

8f56fd3

Pass updates for single frequencies to all clients

c8659e5

Update code comment

aab7025

Remove unused import

46964dc

Clean up names

1bac815

Fix up onSpeaker

efd8dc1

Refactor

47c4978

Headset

cbfb592

Add xca support to backend

8ec7a3f

Add xca update to UI

609ced3

Revert change

22e7ced

Code cleanup

1b09387