Dial-out with multiple phone attendees
Nobody has claimed this yet.
- Dominant language
- JavaScript
- Stars
- 2.2k
- Forks
- 586
- Avg merge
- 18h 27m
- Merged PRs (30d)
- 333
Description
[!TIP]
How to use GitHub
- Please use the 👍 reaction to show that you are affected by the same issue.
- Please don't comment if you have no relevant information to add. It's just extra noise for everyone subscribed to this issue.
- Subscribe to receive notifications on status change and new comments.
I guess it is intended that way, but I can see it can be confusing.
Steps to reproduce
- Have a dial-out room
- Add another phone attendee
- Start the call
Expected behaviour
Not fully sure, I'd expect that all phone attendees are called? At the same time, in a normal conversation, none are called automatically?
Actual behaviour
Only the first one in the list will be called: https://github.com/nextcloud/spreed/blob/01d1b7b7a2e71910a4b96741af9e438606342e57/src/composables/useJoinCall.ts#L133-L141
Talk app
Talk app version: 34/main
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by reproducing the dial-out flow with a dial-out room and an additional phone attendee. Read src/composables/useJoinCall.ts around lines 133-141 to understand the current selection behavior. Clarify whether all phone attendees, or only one, should be called and document the expected result before deciding what completion means.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- javascript
- Domain
- audio-video-rtc, frontend
- Issue type
- Bug
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Quiet
- Clarity
- Needs clarification
- Newbie friendliness
- 35/100