From f18e7f797fe854b30b8519202cf824efdaed90b5 Mon Sep 17 00:00:00 2001 From: 64JohnLee <64lamei@gmail.com> Date: Tue, 9 Jun 2026 21:48:56 +0800 Subject: [PATCH] fix(run-info): add PERIODTYPE to hardware info dropdown PERIODTYPE (DAE:PERIODTYPE) was missing from the runInfoPVs map that populates the show/hide hardware info dropdown in TopBar. Added it alongside the other period-related PVs (PERIOD, PERIODSEQ, etc.) so it is subscribed via websocket and displayed in the dropdown the same way as all other run information variables. Closes #329 --- app/components/Instrument.ts | 4 ++++ app/components/__snapshots__/TopBar.test.ts.snap | 14 ++++++++++++++ 2 files changed, 18 insertions(+) diff --git a/app/components/Instrument.ts b/app/components/Instrument.ts index 347aeb5..7078a6c 100644 --- a/app/components/Instrument.ts +++ b/app/components/Instrument.ts @@ -151,6 +151,10 @@ export class Instrument { pvaddress: `${this.prefix}DAE:PERIODSEQ`, human_readable_name: "Period Sequence", }, + { + pvaddress: `${this.prefix}DAE:PERIODTYPE`, + human_readable_name: "Period Type", + }, { pvaddress: `${this.prefix}DAE:GOODFRAMES_PD`, human_readable_name: "Period Good Frames", diff --git a/app/components/__snapshots__/TopBar.test.ts.snap b/app/components/__snapshots__/TopBar.test.ts.snap index 7883715..424a7b5 100644 --- a/app/components/__snapshots__/TopBar.test.ts.snap +++ b/app/components/__snapshots__/TopBar.test.ts.snap @@ -573,6 +573,20 @@ exports[`renders topbar unchanged 1`] = ` class="break-all" /> +
+ Period Type + : +
+ + +