WordPress.org

Making WordPress.org

Opened 4 years ago

Last modified 16 months ago

#987 assigned enhancement

Display WPTV videos on speaker and producer profiles

Reported by: iandunn Owned by:
Milestone: Priority: normal
Component: Profiles Keywords:
Cc:

Description

Per today's Community Team meeting -- 1, 2 -- we'd like to display WPTV activity on the profiles of the people presenting in the video, and the person who edited the video.

For each new video that is published on WordPress.TV, their profiles should get a new item in their activity stream, and a new item in their corresponding videos section. Both items should link to the video on WPTV.

The videos sections would be laid out similar to the current Themes section, and would display the thumbnail. There would be a section for videos the user is speaking in, and a separate section for videos they edited.

On the WordPress.tv side

  • When a video is submitted, collect the WordPress.org usernames for the speaker(s) and producer. This is being done for the producer in #986, but will need to be done in this ticket for speakers. Some videos can have multiple speakers (e.g, panel discussions), so we need to collect the usernames for each. WPTV already has a taxonomy for speakers, though, so also storing their names/usernames in postmeta feels kind of icky. I'm not sure of a good solution to that off the top of my head. Maybe having them in both the metadata and the taxonomy isn't so bad?
  • When a new video is published, send activity notifications to Profiles for all the speakers and the producer. The payload would include the video title and URL.
  • Extend the WordPress.com REST API so that so that Profiles can pull all the videos that a given WordPress.org username has spoken in or edited. The the /sites/wordpress.tv/posts/ endpoint provides most of what we want, but we'll need to be able to query it by the username meta key, which will be private, so it won't be exposed to the API by default. The response will also need to expose the thumbnail URL (provided by VideoPress), and the event name.
  • It'd be nice to backfill the existing WPTV posts with the speaker and producer usernames. If people are willing to collect all the data into a CSV, we could have a script handle the bulk update and notifications. That's just a nice-to-have, though, not a requirement.

On the Profiles side

  • Add an activity handler for WordPress.tv
  • Periodically and asynchronously pull video data for each user from the WPTV API and cache it in usermeta
  • Add new sections for videos the user spoke in, and videos they produced.

Change History (9)

#1 @iandunn
4 years ago

  • Owner set to iandunn
  • Status changed from new to accepted

#2 @iandunn
4 years ago

Profiles will also need to be updated to accept notification payloads from WPTV, since it's not hosted on the WordPress.org infrastructure.

We wouldn't want to just use the WordPress.com IP blocks, though, because then other sites on WordPress.com could send malicious payloads. There's probably no risk of that right now because all of their client's code is reviewed before being deployed, but that could hypothetically change in the future.

Maybe checking the IP and a token would be enough.

This ticket was mentioned in Slack in #outreach by iandunn. View the logs.


4 years ago

#4 @BrashRebel
4 years ago

  • Cc kyle@… added

This ticket was mentioned in Slack in #outreach by iandunn. View the logs.


4 years ago

#6 @SergeyBiryukov
3 years ago

  • Keywords needs-patch removed

Removing 'needs-patch' from components that are not open sourced yet, to help focus on tickets that actually need a patch.

This ticket was mentioned in Slack in #meta by ocean90. View the logs.


3 years ago

#8 @melchoyce
19 months ago

Love this idea!

#9 @iandunn
16 months ago

  • Owner iandunn deleted
  • Status changed from accepted to assigned
Note: See TracTickets for help on using tickets.