Skip to content

Propagate final to the redeclared element parameters - #606

Open
AntoineGautier wants to merge 5 commits into
mainfrom
ag/605-final-elements
Open

Propagate final to the redeclared element parameters #606
AntoineGautier wants to merge 5 commits into
mainfrom
ag/605-final-elements

Conversation

@AntoineGautier

@AntoineGautier AntoineGautier commented Aug 25, 2026

Copy link
Copy Markdown
Collaborator

Description

This addresses #605.
It includes the changes from #604.

Testing

Unit tests added: npx jes-t "Propagates 'final'"

End-to-end tests have been performed by creating the SOO document with the following system options.

AHU

  • Separate dampers w/ dp
  • Modulating relief w/o fan
  • No heating coil
  • Freeze stat hardwired to both

Cooling-only box

  • 3 sensors = true

✅ The UI behavior matches the current deployed version.
✅ The feature branch produces the same document as the main branch.

@AntoineGautier
AntoineGautier marked this pull request as draft August 25, 2026 18:48
@AntoineGautier
AntoineGautier marked this pull request as ready for review August 26, 2026 07:32
@AntoineGautier AntoineGautier changed the title Propagate final redeclare to the element parameters Propagate final to the redeclared element parameters Aug 26, 2026
@JayHuLBL

Copy link
Copy Markdown
Collaborator

@AntoineGautier Can you address the conflict?

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.

2 participants