Merge lp:~cjohnston/summit/refresh-hangout-link into lp:summit

[Chris Johnston]

Refresh hangout link when the broadcast url is added

[Adnane Belmadiaf (daker)]

+1 from me

review: Approve

[Michael Hall (mhall119)]

Doesn't work

review: Needs Fixing

[Adnane Belmadiaf (daker)]

i think it should be something like :
$('#hangout_url').html('<h3><a href="'+ meeting.hangout_url +'">Join the Hangout on Air</a></h3>');
since we need to get the link from json object not the python object

[Michael Hall (mhall119)]

Except you're not going to get meeting.hangout_url from the JSON API because we filter that out so that it isn't publicly available, and we can't filter based on an instance of the data being returned ATM

Unmerged revisions

497. By Chris Johnston on 2013-03-04

Refresh join hangout link

=== modified file 'summit/schedule/templates/schedule/virtual_meeting.html'
--- summit/schedule/templates/schedule/virtual_meeting.html	2013-03-04 17:47:31 +0000
+++ summit/schedule/templates/schedule/virtual_meeting.html	2013-03-04 21:02:21 +0000
@@ -91,9 +91,11 @@
         {% endif %}
         <div id="description">
             {{ meeting.description|markdown:'safe' }}
+            <div id="hangout_url">
             {% if required and meeting.hangout_url %}
                 <h3><a href="{{ meeting.hangout_url }}">Join the Hangout on Air</a></h3>
             {% endif %}
+            </div>
         </div>
     </section>
 </div>
@@ -112,6 +114,9 @@
                       function(meeting) {
                         if (meeting.broadcast_url != '' && meeting.broadcast_url != null) {
                             $('#broadcast_video').html('<iframe style="width:100%;" height="395" src="'+meeting.broadcast_url+'" frameborder="0" allowfullscreen></iframe>');
+                            {% if required %}
+                                $('#hangout_url').html('<h3><a href="{{ meeting.hangout_url }}">Join the Hangout on Air</a></h3>');
+                            {% endif %}
                         } else {
                             setTimeout(update_video, 20000);
                         }