fix(calls): starting a call in a voice channel no longer @room-pings everyone (#295) #378

Merged
jared merged 3 commits from fix/voice-room-no-room-ping into lotus 2026-10-17 20:38:38 -04:00
Owner

Fixes #295. This is also the root cause of the "missed call from me" seen in voice channels.

CallEmbed asked Element Call to send a call-start notification for every non-DM call. In persistent voice channels and space rooms that notification carries an @room mention:

  • every member got a highlighted ping (and a phone push) whenever a moderator or admin joined first;
  • other clients showed it as a missed call.

Change

  • Only rooms that can ring (isRingableRoom: DMs, and invite-only groups outside a space) get ring / notification. Everything else passes sendNotificationType=none.
  • Pin @lotusguild/element-call-embedded 0.25.0-lotus.23, which adds none (fork PR element-call#48).

Verified with the published 0.25.0-lotus.23, an admin joining the Voice Lounge:

  • Before: sendNotificationType=notification, and Bob got 1 new highlighted notification.
  • After: sendNotificationType=none, and Bob got none.

DMs still send ring; that logic is unchanged and covered by the isRingableRoom tests.

tsc clean · unit tests pass · ESLint clean.

🤖 Generated with Claude Code

https://claude.ai/code/session_017PCb59uY3UKqDqt7B6hzd5

Fixes #295. This is also the root cause of the "missed call from me" seen in voice channels. `CallEmbed` asked Element Call to send a call-start notification for **every** non-DM call. In persistent voice channels and space rooms that notification carries an **@room** mention: - every member got a highlighted ping (and a phone push) whenever a moderator or admin joined first; - other clients showed it as a missed call. **Change** - Only rooms that can ring (`isRingableRoom`: DMs, and invite-only groups outside a space) get `ring` / `notification`. Everything else passes `sendNotificationType=none`. - Pin `@lotusguild/element-call-embedded` **0.25.0-lotus.23**, which adds `none` (fork PR element-call#48). **Verified** with the published 0.25.0-lotus.23, an admin joining the Voice Lounge: - **Before:** `sendNotificationType=notification`, and Bob got 1 new highlighted notification. - **After:** `sendNotificationType=none`, and Bob got **none**. DMs still send `ring`; that logic is unchanged and covered by the `isRingableRoom` tests. `tsc` clean · unit tests pass · ESLint clean. 🤖 Generated with [Claude Code](https://claude.com/claude-code) https://claude.ai/code/session_017PCb59uY3UKqDqt7B6hzd5
jared added 3 commits 2026-10-17 18:42:54 -04:00
CallEmbed asked Element Call to send a call-start notification for every
non-DM call. In persistent voice channels and space rooms it carries an
@room mention: every member got a highlighted ping (and phone push)
whenever a moderator or admin joined first, and other clients showed it
as a missed call. Now only rooms that can ring (isRingableRoom: DMs and
invite-only groups outside a space) get ring/notification; everything
else passes sendNotificationType=none (element-call-embedded 0.25.0-lotus.23).

Verified with a local build of the fork: an admin joining the Voice Lounge
sends none, and Bob gets no new notification (before: +1 highlighted).

Fixes #295

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_017PCb59uY3UKqDqt7B6hzd5
chore(call): pin element-call-embedded 0.25.0-lotus.23 (sendNotificationType=none, #295)
CI / Build & Quality Checks (pull_request) Successful in 2m48s
CI / Trigger Desktop Build (pull_request) Skipped
CI / Docker image build & smoke test (pull_request) Skipped
CI / Secret scan (gitleaks) (pull_request) Successful in 10s
CI / Playwright smoke (e2e) (pull_request) Successful in 15m7s
d7ea408fd0
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_017PCb59uY3UKqDqt7B6hzd5
jared merged commit 906a034e26 into lotus 2026-10-17 20:38:38 -04:00
Sign in to join this conversation.
No Reviewers
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: LotusGuild/cinny#378