Merge lp:~jamalta/summit/793018-update-meeting-summary into lp:summit

Proposed by Jamal Fanaian on 2011-09-16
Status: Merged
Approved by: Michael Hall on 2011-09-16
Approved revision: 172
Merged at revision: 177
Proposed branch: lp:~jamalta/summit/793018-update-meeting-summary
Merge into: lp:summit
Diff against target: 12 lines (+2/-0)
1 file modified
summit/schedule/models/meetingmodel.py (+2/-0)
To merge this branch: bzr merge lp:~jamalta/summit/793018-update-meeting-summary
Reviewer Review Type Date Requested Status
Michael Hall (community) 2011-09-16 Approve on 2011-09-16
Review via email: mp+75793@code.launchpad.net

Commit message

Updating the description of a meeting from the LP blueprint.

Description of the change

Updating the description of a meeting from the LP blueprint.

To post a comment you must log in.
Michael Hall (mhall119) wrote :

I think we're going to need to introduce Mock to start testing things like that that make calls out to Launchpad.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'summit/schedule/models/meetingmodel.py'
2--- summit/schedule/models/meetingmodel.py 2011-09-15 20:36:44 +0000
3+++ summit/schedule/models/meetingmodel.py 2011-09-16 18:07:27 +0000
4@@ -225,6 +225,8 @@
5 data = json.load(lpdata)
6 if 'title' in data:
7 self.title = data['title'][:100]
8+ if 'summary' in data:
9+ self.description = data['summary']
10 except (urllib2.HTTPError, KeyError):
11 pass
12 else:

Subscribers

People subscribed via source and target branches