Merge lp:~markjtully/gwibber/foursquareapi20120612-precise into lp:gwibber/3.4

Proposed by Mark Tully on 2012-06-12
Status: Merged
Merged at revision: 1320
Proposed branch: lp:~markjtully/gwibber/foursquareapi20120612-precise
Merge into: lp:gwibber/3.4
Diff against target: 30 lines (+3/-3)
1 file modified
gwibber/microblog/plugins/foursquare/__init__.py (+3/-3)
To merge this branch: bzr merge lp:~markjtully/gwibber/foursquareapi20120612-precise
Reviewer Review Type Date Requested Status
Ken VanDine 2012-06-12 Approve on 2012-06-27
Review via email: mp+109942@code.launchpad.net

Description of the change

Fixes breakage due to recent changes to the Foursquare API.

To post a comment you must log in.
Ken VanDine (ken-vandine) wrote :

Looks good, thanks!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'gwibber/microblog/plugins/foursquare/__init__.py'
2--- gwibber/microblog/plugins/foursquare/__init__.py 2012-02-19 02:02:27 +0000
3+++ gwibber/microblog/plugins/foursquare/__init__.py 2012-06-12 21:31:24 +0000
4@@ -9,7 +9,7 @@
5
6 PROTOCOL_INFO = {
7 "name": "Foursquare",
8- "version": "4.0",
9+ "version": "5.0",
10
11 "config": [
12 "private:secret_token",
13@@ -102,7 +102,7 @@
14 m["sender"] = {}
15 m["sender"]["id"] = data["user"]["id"]
16 m["sender"]["image"] = data["user"]["photo"]
17- m["sender"]["url"] = data["user"]["canonicalUrl"]
18+ m["sender"]["url"] = "https://www.foursquare.com/user/" + data["user"]["id"]
19 if data["user"]["relationship"] == "self":
20 m["sender"]["is_me"] = True
21 else:
22@@ -148,7 +148,7 @@
23 "id" : comment["user"]["id"],
24 "is_me": False,
25 "image": comment["user"]["photo"],
26- "url" : comment["user"]["canonicalUrl"]
27+ "url" : "https://www.foursquare.com/user/" + comment["user"]["id"]
28 }
29 # Create a comment
30 m["comments"].append({

Subscribers

People subscribed via source and target branches

to all changes: