dohertj2/ScadaBridge
1
0
Fork 0
Code Issues Pull Requests 1 Actions Packages Projects Releases Wiki Activity

docs(cli): note intentional omission of HiLo per-setpoint priorities/deadbands/messages (review fix)

Browse Source
This commit is contained in:
Joseph Doherty
2026-06-07 10:06:58 -04:00
parent f0b144ebda
commit c84eb5aeef
1 changed files with 3 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/ZB.MOM.WW.ScadaBridge.CLI/AlarmTriggerConfigJson.cs
+3
View File
@@ -56,6 +56,9 @@ internal static class AlarmTriggerConfigJson
w.WriteString("direction", NormalizeDirection(direction)); w.WriteString("direction", NormalizeDirection(direction));
break; break;
case "hilo": case "hilo":
// Only the four setpoints are exposed as flags. The codec also accepts
// per-setpoint priorities/deadbands/messages — intentionally omitted here;
// use raw --trigger-config for those (see the YAGNI scope guard in the plan).
if (loLo.HasValue) w.WriteNumber("loLo", loLo.Value); if (loLo.HasValue) w.WriteNumber("loLo", loLo.Value);
if (lo.HasValue) w.WriteNumber("lo", lo.Value); if (lo.HasValue) w.WriteNumber("lo", lo.Value);
if (hi.HasValue) w.WriteNumber("hi", hi.Value); if (hi.HasValue) w.WriteNumber("hi", hi.Value);