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

AdminUI /raw: decide auto-created "Device1", live-verify first-click fix, add a browsePageDelayMs field #596

New Issue
Closed
opened 2026-10-08 02:31:09 -04:00 by dohertj2 · 1 comment
dohertj2 commented 2026-10-08 02:31:09 -04:00
Owner
Copy Link
Copy Source

Left open by #577/#572.
Sources: closing comment on #577; docs/drivers/AbCip.md; src/Server/ZB.MOM.WW.OtOpcUa.AdminUI/Components/Shared/Raw/RawTree.razor:15,297,425-426.

  • Product decision: should creating a driver auto-create an empty Device1 (pinned by RawTreeServiceWaveBTests)? Implement or record as by-design
  • Live-verify on docker-dev that the first click after page load is handled (prerender:false fix; the automation tab was hidden during the 2026-10-08 check)
  • Add a browsePageDelayMs field to the AbCip driver form (today JSON-only)
  • Remove the stale "Browse device is a placeholder (Wave C)" comments in RawTree.razor (OnBrowseDevice is implemented)

Filed 2026-10-08 by the pending/deferred-work sweep after the #562–#591 issue sweep.

Left open by #577/#572. Sources: closing comment on #577; docs/drivers/AbCip.md; src/Server/ZB.MOM.WW.OtOpcUa.AdminUI/Components/Shared/Raw/RawTree.razor:15,297,425-426. - [ ] Product decision: should creating a driver auto-create an empty `Device1` (pinned by `RawTreeServiceWaveBTests`)? Implement or record as by-design - [ ] Live-verify on docker-dev that the first click after page load is handled (`prerender:false` fix; the automation tab was hidden during the 2026-10-08 check) - [ ] Add a `browsePageDelayMs` field to the AbCip driver form (today JSON-only) - [ ] Remove the stale "Browse device is a placeholder (Wave C)" comments in RawTree.razor (`OnBrowseDevice` is implemented) _Filed 2026-10-08 by the pending/deferred-work sweep after the #562–#591 issue sweep._
dohertj2 added the driver/abcip label 2026-10-08 02:31:09 -04:00
dohertj2 commented 2026-10-08 09:24:29 -04:00
Author
Owner
Copy Link
Copy Source

Resolved on master (merge 0fb194a0; R3 follow-up 1475991f, merge 85491f1e).

  • Device1: recorded as by design (docs/Raw.md "Creating a driver auto-creates an empty Device1"). The tag/driver model needs at least one Device carrying the endpoint, so an empty renamable, deletable Device1 is the lowest-friction start; still pinned by RawTreeServiceWaveBTests.
  • First-click: live-verified on the docker-dev rig at a702875d. The earlier miss was automation focus on a hidden tab; both a JS click and a screenshot-first click expand the node on the first click.
  • AbCip form: "Browse page delay (ms)" field added (AbCipDriverForm.razor, browsePageDelayMs), omitted from the JSON when left at default; round-trip test in AbCipDriverPageFormSerializationTests (9c39f2b6, 1475991f).
  • Stale "Browse device is a placeholder (Wave C)" comments removed from RawTree.razor (9c39f2b6).

Closed 2026-10-08 by the hardware-available issue sweep (master d898262c).

Resolved on master (merge 0fb194a0; R3 follow-up 1475991f, merge 85491f1e). - [x] Device1: recorded as **by design** (docs/Raw.md "Creating a driver auto-creates an empty Device1"). The tag/driver model needs at least one Device carrying the endpoint, so an empty renamable, deletable Device1 is the lowest-friction start; still pinned by RawTreeServiceWaveBTests. - [x] First-click: live-verified on the docker-dev rig at a702875d. The earlier miss was automation focus on a hidden tab; both a JS click and a screenshot-first click expand the node on the first click. - [x] AbCip form: "Browse page delay (ms)" field added (`AbCipDriverForm.razor`, `browsePageDelayMs`), omitted from the JSON when left at default; round-trip test in AbCipDriverPageFormSerializationTests (9c39f2b6, 1475991f). - [x] Stale "Browse device is a placeholder (Wave C)" comments removed from RawTree.razor (9c39f2b6). *Closed 2026-10-08 by the hardware-available issue sweep (master d898262c).*
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
auto-managed
queue management
 cross-driver
queue management
 driver/abcip
queue management
 driver/ablegacy
queue management
 driver/focas
queue management
 driver/mqtt
MQTT / Sparkplug B broker-subscribe driver
 driver/opcuaclient
queue management
 driver/s7
queue management
 driver/twincat
queue management
 phase/1
queue management
 phase/2
queue management
 phase/3
queue management
 phase/4
queue management
 phase/5
queue management
 phase/6
queue management
 queue/blocked
queue management
 queue/done
queue management
 queue/failed
queue management
 queue/in-progress
queue management
 queue/queued
queue management
No Label driver/abcip
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
dohertj2 (dohertj2)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: dohertj2/lmxopcua#596
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?