Quam Plures

Merge lp://staging/~quam-plures-core/quam-plures/youtube-iframe into lp://staging/quam-plures

  1. youtube-iframe
  2. Merge into trunk
Proposed by Yabs
Status: Merged
Merged at revision: 7601
Proposed branch: lp://staging/~quam-plures-core/quam-plures/youtube-iframe
Merge into: lp://staging/quam-plures
Diff against target: 33 lines (+7/-1)
1 file modified
qp_plugins/video_plugin/video_modules/_youtube.vp.php (+7/-1)
To merge this branch: bzr merge lp://staging/~quam-plures-core/quam-plures/youtube-iframe
Reviewer Review Type Date Requested Status
EdB Approve
Review via email: mp+58897@code.staging.launchpad.net

Description of the change

Users can now paste iframe embed code

¥

To post a comment you must log in.
Revision history for this message
Kimberly (kimberly-netweb360) wrote :

Tested and working; thanks.

Revision history for this message
EdB (edb) wrote :

downloading but probably won't be able to test for a while :(

Revision history for this message
Yabs (yabs) wrote :

No worries, I'm not exactly over endowed with coding time for QP myself ;)

¥

Revision history for this message
Yabs (yabs) wrote :

Ooops, sorry Kimberly I missed your comment :-S

Thanks for testing :D

¥

Revision history for this message
EdB (edb) wrote :

Sorry for the delay, but finally tested and approving and will merge into core tomorrow (just in case anyone else wants to wait till the last minute), but I did notice a funky thing that I'll bring up in the forums. If I can find the thread :(

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches