Merge lp:~cjohnston/loco-team-portal/edge into lp:loco-team-portal

Proposed by Chris Johnston
Status: Merged
Approved by: Michael Hall
Approved revision: 384
Merged at revision: 390
Proposed branch: lp:~cjohnston/loco-team-portal/edge
Merge into: lp:loco-team-portal
Diff against target: 16 lines (+2/-2)
1 file modified
loco_directory/common/launchpad.py (+2/-2)
To merge this branch: bzr merge lp:~cjohnston/loco-team-portal/edge
Reviewer Review Type Date Requested Status
Michael Hall (community) Approve
Review via email: mp+48709@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Michael Hall (mhall119) wrote :

I made a minor change to use the 1.0 api instead of beta, otherwise it's good.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'loco_directory/common/launchpad.py'
2--- loco_directory/common/launchpad.py 2010-12-21 03:11:18 +0000
3+++ loco_directory/common/launchpad.py 2011-02-05 21:14:27 +0000
4@@ -76,10 +76,10 @@
5 # When this bug is fixed, this can be made cleaner.
6 try:
7 lp.people[identity].mugshot.open()
8- return "https://edge.launchpad.net/api/beta/~%s/mugshot" % (identity)
9+ return "https://launchpad.net/api/beta/~%s/mugshot" % (identity)
10 except HTTPError:
11 # 404 or some other issue that means we should default to False
12- return "https://edge.launchpad.net/api/beta/ubuntu/mugshot"
13+ return "https://launchpad.net/api/beta/ubuntu/mugshot"
14
15 def get_user_timezone(username, lp=None):
16 timezone = 'UTC'

Subscribers

People subscribed via source and target branches