Skip to content

Also pause music on participant-joined#29

Merged
stephendolan merged 4 commits intotupleapp:mainfrom
sambostock:pause-on-participant-joined
Aug 7, 2025
Merged

Also pause music on participant-joined#29
stephendolan merged 4 commits intotupleapp:mainfrom
sambostock:pause-on-participant-joined

Conversation

@sambostock
Copy link
Copy Markdown
Contributor

The existing triggers pause music (Apple Music & Spotify, respectively) on call-started. However, a user can join a Room and be waiting for a while for other participants to join, and may choose to resume their music. Therefore, it makes sense to also pause music on participant-joined.

The existing triggers pause music (Apple Music & Spotify, respectively)
on `call-started`. However, a user can join a Room and be waiting for a
while for other participants to join, and may choose to resume their
music. Therefore, it makes sense to also pause music on
`participant-joined`.
@Crown0815
Copy link
Copy Markdown
Contributor

Hi @sambostock,
just trying to understand whether this would also be useful for #27.

It tested that if I am alone in a room, the music already keeps playing. When somebody joins the music stops.

Is this change to handle a situation where I am listening to music while another person in the room already?

@stephendolan
Copy link
Copy Markdown
Contributor

@Crown0815 Thanks for the PR! If you merge the latest changes from main into your branch and update this PR, the validations should run appropriately and we can get this reviewed 😄

@github-actions
Copy link
Copy Markdown

github-actions bot commented Aug 6, 2025

Thanks for your contribution, @sambostock! 🎉

We've done some automated sense checks on your triggers. All looks great! Someone from Tuple will now manually review your triggers' code and get back to you.

pause-music

  • All good! ✅

pause-spotify

  • All good! ✅

@stephendolan stephendolan added claude-review Start an automated Claude Code review. and removed claude-review Start an automated Claude Code review. labels Aug 6, 2025
@stephendolan stephendolan added claude-review Start an automated Claude Code review. and removed claude-review Start an automated Claude Code review. labels Aug 7, 2025
@stephendolan stephendolan added the claude-review Start an automated Claude Code review. label Aug 7, 2025
@stephendolan stephendolan merged commit 21f6ef2 into tupleapp:main Aug 7, 2025
2 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

claude-review Start an automated Claude Code review.

Development

Successfully merging this pull request may close these issues.

3 participants