Skip to content

feat: add permanent call rooms frontend - #17332

Merged
DorraJaouad merged 9 commits into
mainfrom
fix/noid/voice-rooms-frontend
Apr 9, 2026
Merged

feat: add permanent call rooms frontend#17332
DorraJaouad merged 9 commits into
mainfrom
fix/noid/voice-rooms-frontend

Conversation

@DorraJaouad

@DorraJaouad DorraJaouad commented Mar 10, 2026

Copy link
Copy Markdown
Contributor

☑️ Resolves

🖌️ UI Checklist

🖼️ Screenshots / Screencasts

🏚️ Main view 🏡 New conversation
image image

🚧 Tasks

  • Breakout rooms should be forbidden from backend

🏁 Checklist

  • 🌏 Tested with different browsers / clients:
    • Chromium (Chrome / Edge / Opera / Brave)
    • Firefox
    • Safari
    • Talk Desktop
    • Integrations with Files sidebar and other apps
    • Not risky to browser differences / client
  • 🖌️ Design was reviewed, approved or inspired by the design team
  • ⛑️ Tests are included or not possible
  • 📗 User documentation in https://github.com/nextcloud/documentation/tree/master/user_manual/talk has been updated or is not required

@DorraJaouad DorraJaouad added this to the 🏖️ Next Major (34) milestone Mar 10, 2026
@DorraJaouad
DorraJaouad requested a review from Antreesy March 10, 2026 14:47
@DorraJaouad DorraJaouad self-assigned this Mar 10, 2026
Comment thread lib/Service/AvatarService.php Outdated
Comment thread src/constants.ts
Comment thread src/components/NewConversationDialog/NewConversationDialog.vue Outdated
Comment thread src/components/NewConversationDialog/NewConversationSetupPage.vue Outdated
Comment thread src/components/NewConversationDialog/NewConversationSetupPage.vue Outdated
Comment thread src/components/ConversationIcon.vue Outdated
Comment thread src/components/ConversationSettings/ConversationSettingsDialog.vue Outdated
Comment thread src/components/ConversationSettings/ConversationSettingsDialog.vue
Comment thread src/components/TopBar/CallButton.vue Outdated
Comment thread src/App.vue Outdated
@DorraJaouad
DorraJaouad force-pushed the fix/noid/voice-rooms-frontend branch 2 times, most recently from c7c16bc to 13f01e7 Compare March 23, 2026 12:30

@Antreesy Antreesy left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Otherwise no objections, clean and simple. Let's wait for design feedback!

Two cents from me to discuss:

  • when you're the first joined, shoud the waiting music be playing?
  • when you're joining a 'voice room', should camera be enabled?

Comment thread src/App.vue
Comment thread src/components/TopBar/CallButton.vue
@DorraJaouad
DorraJaouad force-pushed the fix/noid/voice-rooms-frontend branch from 13f01e7 to ac209db Compare April 3, 2026 12:50
Comment thread src/components/NewConversationDialog/NewConversationSetupPage.vue Outdated
Comment thread src/components/NewConversationDialog/NewConversationSetupPage.vue Outdated
Comment thread src/components/NewConversationDialog/NewConversationSetupPage.vue Outdated
Comment thread src/components/NewConversationDialog/NewConversationSetupPage.vue
Comment thread src/components/NewConversationDialog/NewConversationSetupPage.vue Outdated
Comment thread src/components/NewConversationDialog/NewConversationSetupPage.vue Outdated
Comment thread img/icon-voice-room-white.svg Outdated
Comment thread src/App.vue Outdated
@DorraJaouad
DorraJaouad force-pushed the fix/noid/voice-rooms-frontend branch from ac209db to 2c84bea Compare April 8, 2026 08:48
Comment thread src/components/ConversationIcon.vue Outdated
Comment thread lib/RoomPresets/VoiceRoom.php Outdated
@DorraJaouad
DorraJaouad force-pushed the fix/noid/voice-rooms-frontend branch 2 times, most recently from 8c0578b to 1817ba7 Compare April 8, 2026 19:21

@Antreesy Antreesy left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

integration test failing is valid, frontend part is good.

Since you're rebasing anyway - please make joinCallAutomatically a separate commit. I'd like to backport it, feels like it can be useful in solving the breakout/extended rooms issue

@DorraJaouad
DorraJaouad force-pushed the fix/noid/voice-rooms-frontend branch from 1817ba7 to 83a0589 Compare April 9, 2026 13:04
Signed-off-by: Dorra Jaouad <dorra.jaoued7@gmail.com>
Signed-off-by: Dorra Jaouad <dorra.jaoued7@gmail.com>
Signed-off-by: Dorra Jaouad <dorra.jaoued7@gmail.com>
Signed-off-by: Dorra Jaouad <dorra.jaoued7@gmail.com>
Signed-off-by: Dorra Jaouad <dorra.jaoued7@gmail.com>
Signed-off-by: Dorra Jaouad <dorra.jaoued7@gmail.com>
Signed-off-by: Dorra Jaouad <dorra.jaoued7@gmail.com>
Signed-off-by: Dorra Jaouad <dorra.jaoued7@gmail.com>
Signed-off-by: Dorra Jaouad <dorra.jaoued7@gmail.com>
@DorraJaouad
DorraJaouad force-pushed the fix/noid/voice-rooms-frontend branch from 83a0589 to be7a297 Compare April 9, 2026 13:06
@DorraJaouad
DorraJaouad merged commit 60ac760 into main Apr 9, 2026
79 checks passed
@DorraJaouad
DorraJaouad deleted the fix/noid/voice-rooms-frontend branch April 9, 2026 13:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

🔃 Permanent call rooms

2 participants