Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Webhook for stream title change #2795

Closed
Chippit opened this issue Mar 11, 2023 · 5 comments · Fixed by #2823
Closed

Webhook for stream title change #2795

Chippit opened this issue Mar 11, 2023 · 5 comments · Fixed by #2823
Assignees
Labels
3rd party integration Related to APIs for inbound/outbound interfacing with others. backlog Ideas that might be cool and can be looked into later. go backend Server-side code written in Go

Comments

@Chippit
Copy link

Chippit commented Mar 11, 2023

Share your bug report, feature request, or comment.

Currently there is no webhook able to respond to stream title changes. This information is included in the webhooks for starting and ending a stream, but if the title changes while a stream is running, a third party app has no way of knowing.

@gabek
Copy link
Member

gabek commented Mar 11, 2023

Seems like a reasonable thing to add.

@gabek gabek added backlog Ideas that might be cool and can be looked into later. 3rd party integration Related to APIs for inbound/outbound interfacing with others. go backend Server-side code written in Go labels Mar 11, 2023
@Pranav2612000
Copy link
Contributor

Hey @gabek I'll like to work on this issue.
I'm guessing we already have webhooks for starting and ending the stream, I can refer those on how the webhook is to be created right? Any other docs or code you think is relevant?

@gabek
Copy link
Member

gabek commented Mar 13, 2023

Awesome, thank you! And yeah, there's a handful of webhooks that already exist. Here is the stream status webhook that already sends along the stream title and here's the documentation around the webhooks that already exist and would need to be updated with a new one.

@luckydonald
Copy link

luckydonald commented Mar 26, 2023

Probably they should be fired for streamTitle and summary changes as well.

@gabek
Copy link
Member

gabek commented Mar 26, 2023

This is for the stream title. I don't think it should be fired for summary, that's not a value that's seen as an attribute of the current stream, that's just the description of the server itself. A separate server metadata webhook could be created, but that's not what this is.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3rd party integration Related to APIs for inbound/outbound interfacing with others. backlog Ideas that might be cool and can be looked into later. go backend Server-side code written in Go
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants