Skip to content

Fix kitchen monitor autosave and CSP theme script#119

Open
osswald wants to merge 2 commits into
mainfrom
fix/kitchen-monitor-autosave
Open

Fix kitchen monitor autosave and CSP theme script#119
osswald wants to merge 2 commits into
mainfrom
fix/kitchen-monitor-autosave

Conversation

@osswald

@osswald osswald commented Jul 9, 2026

Copy link
Copy Markdown
Owner

Summary

  • Include kitchenMonitors in the event configuration autosave watch source so adding or editing kitchen monitors triggers a save (fixes production issue where kitchen monitor changes were never persisted).
  • Add a regression test documenting that unwatched snapshot fields do not trigger autosave.
  • Move the theme bootstrap script from inline index.html to public/theme-init.js so it complies with production CSP (script-src 'self').

Test plan

  • cd cloud/frontend && npm test (153 tests pass)
  • On admin.vendiqo.ch, open an event with kitchen monitors enabled, add a kitchen monitor printer, confirm autosave completes (~800ms) and persists after reload
  • Confirm browser console no longer shows CSP violation for inline script on page load

Made with Cursor

osswald and others added 2 commits June 25, 2026 20:36
Production CSP blocks inline scripts; load theme-init.js from self instead so dark/light mode applies without console violations.

Co-authored-by: Cursor <cursoragent@cursor.com>
@cursor

cursor Bot commented Jul 9, 2026

Copy link
Copy Markdown

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

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