Merge lp://staging/~mhall119/summit/convert-youtube-broadcast-urls into lp://staging/summit
Proposed by
Michael Hall
| Status: | Merged |
|---|---|
| Approved by: | Chris Johnston |
| Approved revision: | 497 |
| Merged at revision: | 494 |
| Proposed branch: | lp://staging/~mhall119/summit/convert-youtube-broadcast-urls |
| Merge into: | lp://staging/summit |
| Diff against target: |
56 lines (+21/-2) 3 files modified
summit/schedule/forms.py (+19/-0) summit/schedule/templates/schedule/edit_hangout.html (+1/-1) summit/schedule/templates/schedule/virtual_meeting.html (+1/-1) |
| To merge this branch: | bzr merge lp://staging/~mhall119/summit/convert-youtube-broadcast-urls |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Chris Johnston | Approve | ||
| Adnane Belmadiaf | Approve | ||
|
Review via email:
|
|||
Commit message
Parse submitted values for broadcast URL to allow copy/pasted values from YouTube/G+ Hangouts other than the actual embed url
Description of the change
This makes the form a little more flexible when it comes to the broadcast URL, since the hangout provides the full iframe snippet instead of just the embed url, the person starting the hangout can copy/paste the entire thing into the form and Summit will strip out just the embed url.
To post a comment you must log in.
Revision history for this message
| Chris Johnston (cjohnston) | # |
review:
Approve
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
+1 from me