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

Email speakers when videos are published #1259

Open
russss opened this issue Sep 1, 2022 · 11 comments
Open

Email speakers when videos are published #1259

russss opened this issue Sep 1, 2022 · 11 comments

Comments

@russss
Copy link
Member

russss commented Sep 1, 2022

It would be good to email speakers a link to their talk video when it's published. Some thoughts:

  • We should probably abstract this somewhere, so we can set talk videos through an internal API (see Video API #1306) or through a command-line tool. (We have some existing video-matching code here but it's a bit brittle).
  • We should only email the speaker the first time this happens (videos may not be uploaded to all video services simultaneously).
  • We should probably also email speakers when the "video lost" flag is set ("Video recording lost" flag on CfP items #1258)
@jayaddison
Copy link
Contributor

jayaddison commented Jun 11, 2024

Some ad-hoc requirements gathering from IRC:

  • On the EMF talk/proposal view (example), add a list of hyperlinks to video URLs below the embedded video object. (done: [proposal view] Add video recording host hyperlinks. #1740)
  • When any non-empty URL -- or the 'video lost' flag -- is first configured for a Proposal, send a one-time email to the speaker with a link to the EMF talk/proposal view URL or an apology message, as appropriate.
    • Safety catch: add an HTTP 4xx response code if+when an HTTP PATCH operation attempts to set both a video URL and also the video-lost flag for a proposal.

@fkusei
Copy link

fkusei commented Jun 12, 2024

I'm currently working on a "generic webhook" feature for our publishing scripts which would maybe help here: voc/voctopublish#109 (feedback welcome)

(Requires changing the API on the website though)

@wlcx
Copy link
Member

wlcx commented Jun 15, 2024

I'm not 100% on emailing the speaker in the case of missing recording - there may be more nuance to it than a generic catch all "something went wrong" message, and in any case I think I'd rather email them with a more personal message and a name attached rather than a "cold" website email.

Of course, that's just me and it's probably better they get something rather than nothing - so maybe the best course is to proceed but also send them a more human email? Just thinking out loud.

@crablab
Copy link
Contributor

crablab commented Jun 16, 2024

Agreed. I don't think we should automatically email people with lost recordings.

We should probably send an alert to the CfP channel instead, so someone from the Content team can reach out directly and discuss with the speaker.

That assumes this is a relatively rare occurrence?

@russss
Copy link
Member Author

russss commented Jun 16, 2024

Historically it has not really been a rare occurrence but I'm hopeful we won't have any lost recordings this year, so I'm happy to skip that for the moment at least.

@russss
Copy link
Member Author

russss commented Sep 1, 2024

Some goalpost-moving, sorry:

  • It looks like we will need a way of sending the emails after the URLs are set. I wonder if it would be best to have this as a command-line task, but we'd need to store the email status somewhere. We could maybe add a "video-published" state to the state machine?
  • We may not make YouTube videos public at the same time that they're uploaded. So I think it's best to make this trigger on the c3voc_url only for the moment.

@Jonty
Copy link
Member

Jonty commented Sep 3, 2024

  • We may not make YouTube videos public at the same time that they're uploaded. So I think it's best to make this trigger on the c3voc_url only for the moment.

If we do this, we should make sure that we include copy in the email telling them that the video will also be published on YouTube at a later date, just so they know this is happening.

@fkusei
Copy link

fkusei commented Sep 3, 2024

In its current implementation, the c3voc webhook will set the youtube url as well, even if the youtube videos are set to private. (There is currently no indication on whether the videos are publicly available, but we can implement that)

@russss
Copy link
Member Author

russss commented Sep 3, 2024

The main considerations here are:

  1. Youtube may refuse to publish a video after upload due to ContentID detection or whatever
  2. It's nicer for people who subscribe to the EMF Youtube channel if all the videos don't get dumped into their feed in quick succession. (Particularly if they happen to be attending EMF at the time.)
  3. It's better for engagement if we publish no more than 3 videos per day because the videos get de-prioritised by youtube after that. (I'm not too concerned about this one, but it does tie in with point 2)

I wonder if we can easily check if the youtube video is public when we send the email? Or c3voc could do that and send us an updated webhook.

Or we just omit the youtube URL from the email and say "Your video will be published on youtube soon", but that still means we'll be linking to a private YT video from our talk pages.

@jayaddison
Copy link
Contributor

  • We may not make YouTube videos public at the same time that they're uploaded. So I think it's best to make this trigger on the c3voc_url only for the moment.

If we do this, we should make sure that we include copy in the email telling them that the video will also be published on YouTube at a later date, just so they know this is happening.

Would applying both of those suggestions in combination provide a complete-enough solution?

Gradual release of YouTube videos to channels would then (I think?) be isolated from the email-notification logic.

@jayaddison
Copy link
Contributor

Gradual release of YouTube videos to channels would then (I think?) be isolated from the email-notification logic.

Sorry, I missed one aspect of the requirements:

Or we just omit the youtube URL from the email and say "Your video will be published on youtube soon", but that still means we'll be linking to a private YT video from our talk pages.

I don't like that I keep missing important details and adding distraction, so I'm going to step back from this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
8 participants