Session's date/time on session info page

[Cheffheid]

A session's speaker name is already displaying on a session's page, but once a session is scheduled why doesn't it display the date/time as well?

It helps to have all that information in one place (as opposed to having to refer back to the schedule page for it) and is especially helpful when you've navigated to an entirely different session with the previous/next links.

[davidjlaietta]

As an example: ​https://2016.orlando.wordcamp.org/session/build-something-today/ Lists the talk title, description, and speaker.

That speaker has been assigned some relevant meta information, most of it visible from ​https://2016.orlando.wordcamp.org/weekend-schedule/

The individual session page could include the link to slides and wordpress.tv video, date and time of session, and track that the session is in.

[iandunn]

+1

[hlashbrooke]

Displaying all session meta data on sinlge session page

[hlashbrooke]

I put together a patch for this that displays all the session meta data on the single session page in one list, after the content. See the screenshots below for an example of how it looks - the meta data includes the speaker(s), date, time, track(s), slides URL, video URL and categories, depending on which of those the session actually has.

[hlashbrooke]

Example listing all available meta data

[hlashbrooke]

Example listing the minimum most commonly used meta data

[davidjlaietta]

I think that this handles it very well.

Replying to hlashbrooke:

I put together a patch for this that displays all the session meta data on the single session page in one list, after the content. See the screenshots below for an example of how it looks - the meta data includes the speaker(s), date, time, track(s), slides URL, video URL and categories, depending on which of those the session actually has.

[hlashbrooke]

Moving meta list into separate view

[hlashbrooke]

After chatting with @iandunn about this, we decided to move the list display to a separate view. The new patch (1921.1.diff) does just that. I used the same method to include the view as elsewhere in the wc-post-types plugin, but let me know if it should eb done differently.

[coreymckrill]

@hlashbrooke The patch is looking good. Two things right now:

• The patch doesn't apply cleanly for me on the latest revision of wc-post-types. Could you check and make sure you've pulled the latest into your dev environment and rebase the patch if necessary?
• I'm seeing some lines commented out that I'm guessing were changed for local testing, but we probably want them to be interpreted on Production:

// if ( $site_id <= apply_filters( 'wcpt_session_post_speaker_info_min_site_id', 463 ) && ! in_array( $site_id, $enabled_site_ids ) ) { 
//      return $content; 
// }

[hlashbrooke]

Updating from master and uncommenting conditional

[hlashbrooke]

All sorted @coreymckrill - rebased from master and I uncommented those lines.

[coreymckrill]

This unfortunately needs to be rebased again @hlashbrooke. Sorry it got overlooked. At this point, I think it would be better to submit this as a pull request to the ​GitHub repo.

[dd32]

This ticket has been moved to GitHub ​https://github.com/WordPress/wordcamp.org/issues/651