Merge lp:~mhall119/summit/preserve-required into lp:summit

Proposed by Chris Johnston
Status: Superseded
Proposed branch: lp:~mhall119/summit/preserve-required
Merge into: lp:summit
Diff against target: 50 lines (+7/-7)
3 files modified
summit/schedule/models/meetingmodel.py (+5/-4)
summit/schedule/tests/tests.py (+1/-2)
summit/schedule/views.py (+1/-1)
To merge this branch: bzr merge lp:~mhall119/summit/preserve-required
Reviewer Review Type Date Requested Status
Chris Johnston Approve
Review via email: mp+150705@code.launchpad.net

This proposal has been superseded by a proposal from 2013-02-27.

Commit message

Keeps required participants required.

To post a comment you must log in.
Revision history for this message
Chris Johnston (cjohnston) wrote :

Conflicts need to be fixed please.

review: Needs Fixing
Revision history for this message
Chris Johnston (cjohnston) :
review: Approve
Revision history for this message
Chris Johnston (cjohnston) :
review: Approve
Revision history for this message
Tarmac WebDev (tarmac-webdev) wrote :

There are additional revisions which have not been approved in review. Please seek review and approval of these new revisions.

Revision history for this message
Chris Johnston (cjohnston) :
review: Approve
Revision history for this message
Tarmac WebDev (tarmac-webdev) wrote :

Attempt to merge into lp:summit failed due to conflicts:

text conflict in summit/schedule/tests/tests.py
text conflict in summit/schedule/views.py

490. By Michael Hall

[r=daker] Adds a javascript refresh to the youtube placeholder image to continue checking for a video url

491. By Chris Johnston

[r=mhall119] Separate tests into different files

492. By Michael Hall

Once again, merge from trunk and fix conflicts

Unmerged revisions

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'summit/schedule/models/meetingmodel.py'
--- summit/schedule/models/meetingmodel.py 2013-02-26 20:00:45 +0000
+++ summit/schedule/models/meetingmodel.py 2013-02-27 14:57:18 +0000
@@ -497,10 +497,11 @@
497 )[0]497 )[0]
498498
499 participant.from_launchpad = True499 participant.from_launchpad = True
500 if required:500 if not participant.participation == 'REQUIRED':
501 participant.participation = 'INTERESTED'501 if required:
502 else:502 participant.participation = 'INTERESTED'
503 participant.participation = 'ATTENDING'503 else:
504 participant.participation = 'ATTENDING'
504 participant.save()505 participant.save()
505 except ObjectDoesNotExist:506 except ObjectDoesNotExist:
506 pass507 pass
507508
=== modified file 'summit/schedule/tests/tests.py'
--- summit/schedule/tests/tests.py 2013-02-27 03:18:38 +0000
+++ summit/schedule/tests/tests.py 2013-02-27 14:57:18 +0000
@@ -3385,7 +3385,7 @@
3385 now = datetime.datetime.now()3385 now = datetime.datetime.now()
3386 week = datetime.timedelta(days=7)3386 week = datetime.timedelta(days=7)
3387 meeting = factory.make_one(Meeting, summit=self.summit, name='test-name', title='Test Title', requires_dial_in=False, private=True)3387 meeting = factory.make_one(Meeting, summit=self.summit, name='test-name', title='Test Title', requires_dial_in=False, private=True)
3388 user = factory.make_one(User, username='testuser', first_name='Test', last_name='User', is_active=True)3388 user = factory.make_one(User, username='testuser', first_name='Test', last_name='User', is_active=True, is_superuser=False)
3389 user.set_password('password')3389 user.set_password('password')
3390 user.save()3390 user.save()
3391 attendee = factory.make_one(Attendee, summit=self.summit, user=user, start_utc=now, end_utc=now+week)3391 attendee = factory.make_one(Attendee, summit=self.summit, user=user, start_utc=now, end_utc=now+week)
@@ -3414,4 +3414,3 @@
3414 response = self.client.get(reverse('summit.schedule.views.search', args=[self.summit.name]), {'q': 'test'})3414 response = self.client.get(reverse('summit.schedule.views.search', args=[self.summit.name]), {'q': 'test'})
3415 self.assertContains(response, self.form_html, 1)3415 self.assertContains(response, self.form_html, 1)
3416 self.assertContains(response, '<a href="%s">Test Title</a>' % meeting.get_meeting_page_url(), 1)3416 self.assertContains(response, '<a href="%s">Test Title</a>' % meeting.get_meeting_page_url(), 1)
3417
34183417
=== modified file 'summit/schedule/views.py'
--- summit/schedule/views.py 2013-02-27 03:18:38 +0000
+++ summit/schedule/views.py 2013-02-27 14:57:18 +0000
@@ -92,7 +92,7 @@
92 for m in matches:92 for m in matches:
93 if not m.private or m.can_view_pm(attendee):93 if not m.private or m.can_view_pm(attendee):
94 meetings.append(m)94 meetings.append(m)
95 95
96 context = {96 context = {
97 'summit': summit,97 'summit': summit,
98 'query': query,98 'query': query,

Subscribers

People subscribed via source and target branches