Skip to content

fix(demo): use bare tag ids for gauge variableId bindings - #2488

Open
sankalpsthakur wants to merge 1 commit into
frangoteam:masterfrom
sankalpsthakur:fix/2486-demo-variable-ids
Open

fix(demo): use bare tag ids for gauge variableId bindings#2488
sankalpsthakur wants to merge 1 commit into
frangoteam:masterfrom
sankalpsthakur:fix/2486-demo-variable-ids

Conversation

@sankalpsthakur

Copy link
Copy Markdown
Contributor

Summary

Demo project used legacy Device^~^Tag variableId form that never matches runtime bare tag ids. Corrected demo bindings to bare tag ids.

Changes

  • Updated all 13 variableId values in client/src/assets/project.demo.fuxap:
    • Device A^~^Tag ByteTag Byte (4 bindings)
    • Device A^~^Tag BoolTag Bool (9 bindings)
  • Left variable / variableSrc and HmiService.toVariableId() unchanged (demo-only, high merge odds).

Validation

  • inspected demo JSON / grep for remaining ^~^ in demo project (0 remaining)
  • parsed demo project as JSON; all 13 variableId values are bare declared tag ids (Tag Byte, Tag Bool) matching devices["Device A"].tags

Fixes #2486

AI/LLM disclosure

  • AI coding tools (including Grok and/or Codex agent-assisted editing) were used to help draft or modify code and this PR description.
  • I reviewed the complete change, understand the reasoning, and ran the reported local tests before submitting.
  • This submission is original work of authorship under the project contributor terms; AI output was not pasted unreviewed.

Demo project bound gauges with Device^~^Tag form that never matches
runtime variable map keys (bare tag ids on the wire). Correct all 13
variableId values to bare tag ids so the demo matches the runtime
contract (frangoteam#2486).
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.

Demo project binds gauges with a variableId form nothing produces (^~^); HmiService.toVariableId() is unused

1 participant