Skip to content

Adding perRoomPrivacyBlur - #1894

Open
Zork130 wants to merge 1 commit into
SableClient:devfrom
Zork130:dev
Open

Adding perRoomPrivacyBlur#1894
Zork130 wants to merge 1 commit into
SableClient:devfrom
Zork130:dev

Conversation

@Zork130

@Zork130 Zork130 commented Aug 20, 2026

Copy link
Copy Markdown

Description

Adds a per-room override for the existing account-wide "blur media" privacy setting. Previously privacyBlur only applied globally across every room; this adds a perRoomPrivacyBlur array ({roomId, blur}[]) that lets a room's blur behavior be explicitly set to "Blur" or "Don't Blur", overriding the global default for that room only, with a "Default" option to fall back to the global setting.

The toggle lives in the room's Cosmetics settings tab (under "Settings"), since the existing "Appearance" tab is space-only and doesn't apply to regular rooms.

Also includes an unrelated one-line Dockerfile fix: bumped the pinned node:24.13.1-alpine base image to 24.15.0-alpine, since jsdom@30 now requires ^24.15.0 and engineStrict: true in pnpm-workspace.yaml made the mismatch fatal — the Docker build was broken before this PR for anyone testing locally.

Fixes #

Type of change

  • New feature (non-breaking change which adds functionality)
  • Bug fix (non-breaking change which fixes an issue)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings

AI disclosure:

  • Partially AI assisted (clarify which code was AI assisted and briefly explain what it does).
  • Fully AI generated (explain what all the generated code does in moderate detail).

…uirement under engineStrict

2. Added a perRoomPrivacyBlur
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