Skip to content

fix: restore control system display name events - #330

Merged
stakach merged 2 commits into
masterfrom
ai/146-display-name-events
Sep 16, 2026
Merged

stakach merged 2 commits into
masterfrom
ai/146-display-name-events

Conversation

@stakach

@stakach stakach commented Sep 16, 2026 •

Copy link
Copy Markdown
Member

ControlSystem display_name changes now notify running services again. Remove only display_name from this model's ignored columns; remaining ControlSystem exclusions and Zone's separate policy are unchanged.

Review and verification

  • Regression uses ordinary saves to set and clear display_name alongside ignored description changes, checking persistence, CDC rows and PostgreSQL notifications.
  • Red: 6 specs, one failure proving the expected update row was suppressed before this change.
  • Remaining metadata/telemetry suppression and runtime update controls retained.
  • README documents explicit replacement of the installed ten-column policy with the corrected nine-column policy, plus earlier-policy alternatives.
  • Independent source/spec/docs review approved. Formatting and Ameba pass (181 files).
  • Focused green: 6 specs, zero failures/errors. Full stable/unstable CI and all PR/style checks passed at e82e54f: https://github.com/PlaceOS/models/actions/runs/35052217743.

Completes the user correction tracked in PlaceOS/local#146.

@github-actions github-actions Bot added the type: bug something isn't working label Sep 16, 2026
@stakach
stakach marked this pull request as ready for review September 16, 2026 03:32
@github-actions github-actions Bot added type: bug something isn't working and removed type: bug something isn't working labels Sep 16, 2026
@stakach
stakach merged commit f1ab5ba into master Sep 16, 2026
8 checks passed
@stakach
stakach deleted the ai/146-display-name-events branch September 16, 2026 03:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: bug something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant