Opened 9 years ago
Last modified 4 years 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 (11)
This ticket was mentioned in Slack in #outreach by iandunn. View the logs.
9 years ago
This ticket was mentioned in Slack in #outreach by iandunn. View the logs.
9 years ago
#6
@
9 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.
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.