Skip to content

Add TF listener and broadcaster in controller base class - #244

Merged
domire8 merged 4 commits into
mainfrom
feat/tf
Apr 10, 2026
Merged

domire8 merged 4 commits into
mainfrom
feat/tf

Conversation

@domire8

@domire8 domire8 commented Apr 7, 2026 •

Copy link
Copy Markdown

Description

Commits from breaking branch (this was another example of us modifying the ABI and then having to revert hehe..)

Review guidelines

Estimated Time of Review: 5 minutes

Checklist before merging:

  • Confirm that the relevant changelog(s) are up-to-date in case of any user-facing changes

Related issues

Blocked by:

@eeberhard eeberhard left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good but of course hold on merging to main until #241 and a 5.4.1 release patch are through

yrh012
yrh012 previously approved these changes Apr 8, 2026

@yrh012 yrh012 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me 👍

SprGrf
SprGrf previously approved these changes Apr 8, 2026
@domire8
domire8 dismissed stale reviews from SprGrf and yrh012 via c824b33 April 9, 2026 09:49
@domire8
domire8 merged commit bcd2c4f into main Apr 10, 2026
4 checks passed
@domire8
domire8 deleted the feat/tf branch April 10, 2026 06:33
@github-actions github-actions Bot locked and limited conversation to collaborators Apr 10, 2026
@domire8 domire8 self-assigned this Apr 14, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants