Skip to content

Mark base state field name as dirty - #38

Merged
johnwduggan merged 1 commit into
mainfrom
base-name
Feb 10, 2026
Merged

johnwduggan merged 1 commit into
mainfrom
base-name

Conversation

@johnwduggan

Copy link
Copy Markdown
Member

Summary of Changes

Very minor bug fix to use the correct name when flushing state variables. The automated testing doesn't detect this since it manually flushes after state changes.

Also, with state being flushed properly, there was an issue with state leaking between each instance of test_binding_trame_to_model since each run was binding to test_object, producing confusing results. I've added an index into each connection name to better isolate each test.

Checklist

  • The PR has a clear and concise title
  • Code is self-documented and follows style guidelines.
  • Automated tests are written and pass successfully.
  • Regression tests (e.g. manually triggered system tests, manual GUI/tool tests, ...) are performed to make sure the PR does not break anything (when applicable)
  • Readme file is present and up-to-date.

Documentation Updates

Additional Notes

@johnwduggan johnwduggan self-assigned this Feb 10, 2026
@johnwduggan
johnwduggan merged commit e70eaf4 into main Feb 10, 2026
1 check passed
@johnwduggan
johnwduggan deleted the base-name branch February 10, 2026 17:14
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.

1 participant