#508 closed enhancement (fixed)
Display links to slides and WPTV video in Session shortcode
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | Priority: | normal | |
Component: | WordCamp Site & Plugins | Keywords: | good-first-bug has-patch |
Cc: |
Description
Displaying links to each session's slides and video in the [session] shortcode output would provide an easy way for WordCamps to organize all the relevant session data and present it to users in a centralized location.
Once the data has been entered, users could visit a page like http://2014.oc.wordcamp.org/sessions/ to get everything related to each session. If slides are entered before the event, users can access the page during sessions if they want to follow along with the presenter. They could also visit it after the event to watch the videos of sessions they missed.
- Add meta field to the Session post type to collect the URL for a session's slides (preferably uploaded to the corresponding WordCamp.org site for archiving, but could potentially be anywhere)
- Add meta field to the Session post type to collect the URL for the video of the presentation on WordPress.tv
- Display those links when generating the output for the [sessions] shortcode, if they've been entered.
Attachments (1)
Change History (16)
This ticket was mentioned in IRC in #wordpress-meta by sams. View the logs.
11 years ago
This ticket was mentioned in Slack in #community by brashrebel. View the logs.
10 years ago
#4
@
10 years ago
Just looking to see if I may be on the right track here as there are multiple ways to tackle this. My current approach has been to add the new fields to the Session Info meta box like so:
I could instead create an additional meta box which would contain these new fields. Not sure whether that would add any value.
Regarding the [session]
shortcode, my first inclination would be to add something like this to the output:
I'm not certain these are the optimal solutions but they will work. I have other ideas for making it easier to use that I may explore. Thanks in advance for any input.
#5
@
10 years ago
That looks great, it's exactly what I was envisioning. If you have an idea for something even better, though, describe it here and we can discuss doing that instead.
The only tweaks I'd suggest to your current approach would be to:
- Change
Video URL
toWordPress.TV URL
, and also to check that the URL submitted is actually on WPTV. It's fine if people want to post their videos to YouTube, etc in addition to WPTV, but WPTV should be the canonical source for all WordCamp sessions. - Make the Session Info metabox appear under the editor instead of in the sidebar, so that the full URLs are visible
#6
@
10 years ago
- Keywords dev-feedback has-patch added
Just added 508.diff which is my first patch submission and I'm not sure I got it right. Should contain all of my changes which include:
- Added slides and video URL fields to the session info meta box
- Moved the session info box below the editor
- Don't save the video URL unless it is from wp.tv
- Added links to the video and slides URLs to the shortcode
- Added
show_links
to the shortcode$atts
and set default totrue
That should be about it. Obviously very open to feedback.
Regarding other ideas, I was toying with the thought of changing the video URL field to a <select>
field instead of an <input type="text">
and populating the <option>
values with a fetch_feed()
call to wp.tv which supplies the year and city to dynamically pull a list of the videos from the appropriate camp if they exist. Might be trickier than it is worth as I'd imagine I would have to parse the get_bloginfo( 'name' )
or something to get the location and stuff like that but it sounded fun to me.
It also occurred to me that if we stored the slides URL in its own field on wp.tv instead of just throwing it in the excerpt, we could potentially retrieve that in the same call we make for the video link to avoid double entry and make this process much easier. WordCamp content moderators could then potentially just edit each session and select the appropriate video (or enter in the URL if selecting would not be feasible) and the video and slides could both be brought in automatically. Although, as you described initially, we should allow for slides to be entered long before the video makes it to wp.tv. Just ideas I guess.
#8
@
10 years ago
- Keywords dev-feedback removed
- Owner set to iandunn
- Status changed from new to accepted
Hey, sorry for the long wait, but this looks really good.
The only question I have is about the new show_links
parameter; I wonder if it's actually necessary. If an organizer didn't want to show the links, then they could just not enter the URLs into the session posts, right?
Is there a use case where they'd want to have the URLs saved, but not display them?
#9
@
10 years ago
Yeah, it probably isn't really necessary. Not sure if I had a use case in mind when I made that but can't think of a need now.
#10
@
10 years ago
Cool, I'll go ahead and leave that out; we can always add it in later if we discover a need for it.
#11
@
10 years ago
It looks like 508.diff
has each commit separated, which doesn't work well because the patch
command will overwrite the .rej
files for each commit.
Do you remember what kind of Git workflow you were using? Maybe it was created with git format-patch
? I tried applying it with git apply
too, but that didn't work. I was able to get it working by manually separating it into 3 files and using patch
on each.
I think git diff master --no-prefix
should generate a patch that's more compatible with SVN.
#13
@
10 years ago
No worries, I haven't tried applying Git patches to SVN before, so it's new to me too :)
I would be interested in participating in this effort if it is still needed.