Merge lp:~justinmcp/unity-webapps-facebookmessenger/app-name-fix-13.10 into lp:unity-webapps-facebookmessenger/13.10

Proposed by Justin McPherson
Status: Merged
Approved by: Alexandre Abreu
Approved revision: 68
Merged at revision: 68
Proposed branch: lp:~justinmcp/unity-webapps-facebookmessenger/app-name-fix-13.10
Merge into: lp:unity-webapps-facebookmessenger/13.10
Diff against target: 32 lines (+3/-3)
2 files modified
FacebookMessenger.test.js (+2/-2)
FacebookMessenger.user.js (+1/-1)
To merge this branch: bzr merge lp:~justinmcp/unity-webapps-facebookmessenger/app-name-fix-13.10
Reviewer Review Type Date Requested Status
Alexandre Abreu (community) Approve
PS Jenkins bot (community) continuous-integration Approve
Review via email: mp+206652@code.launchpad.net

Description of the change

Fixes problem of two facebook launcher items appearing.

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
Alexandre Abreu (abreu-alexandre) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'FacebookMessenger.test.js'
2--- FacebookMessenger.test.js 2013-04-30 20:40:06 +0000
3+++ FacebookMessenger.test.js 2014-02-17 04:35:18 +0000
4@@ -26,7 +26,7 @@
5
6 validateCallLog: function (log, out) {
7 assertEquals('Unity.init', log[0].func, 'Unity.init');
8- assertEquals('facebook', log[0].args[0].name, 'facebook');
9+ assertEquals('FacebookMessenger', log[0].args[0].name, 'FacebookMessenger');
10
11 assertEquals('Unity.init', log[2].func, 'Unity.init');
12 assertEquals('Login', log[2].args[0].login, this._login);
13@@ -41,4 +41,4 @@
14 },
15
16 scriptName: 'FacebookMessenger.user.js'
17-};
18\ No newline at end of file
19+};
20
21=== modified file 'FacebookMessenger.user.js'
22--- FacebookMessenger.user.js 2013-06-17 20:37:11 +0000
23+++ FacebookMessenger.user.js 2014-02-17 04:35:18 +0000
24@@ -90,7 +90,7 @@
25 login = null;
26 }
27
28-Unity.init({ name: "facebook",
29+Unity.init({ name: "FacebookMessenger",
30 iconUrl: "icon://unity-webapps-facebook",
31 login: login,
32 domain: 'facebook.com',

Subscribers

People subscribed via source and target branches

to all changes: