Merge lp:~robert-ancell/unity-webapps-googlecalendar/app-name into lp:unity-webapps-googlecalendar

Proposed by Robert Ancell
Status: Needs review
Proposed branch: lp:~robert-ancell/unity-webapps-googlecalendar/app-name
Merge into: lp:unity-webapps-googlecalendar
Diff against target: 38 lines (+6/-6)
3 files modified
GoogleCalendargooglecom.desktop (+2/-2)
debian/control (+3/-3)
manifest.json (+1/-1)
To merge this branch: bzr merge lp:~robert-ancell/unity-webapps-googlecalendar/app-name
Reviewer Review Type Date Requested Status
PS Jenkins bot (community) continuous-integration Approve
Alexandre Abreu (community) Needs Fixing
Justin McPherson Approve
Review via email: mp+218243@code.launchpad.net

Description of the change

Correct name by adding in space. I'm not sure what XB-Ubuntu-Webapps-Name in debian/control is used for an if that too is a human readable name (i.e. "Google Calendar") or an ID (i.e. "GoogleCalendar").

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
Justin McPherson (justinmcp) :
review: Approve
Revision history for this message
David Barth (dbarth) wrote :

Just verifying with Alex that that name parameter has no implication with technical paths to find the webapp supporting JS code.

Revision history for this message
Alexandre Abreu (abreu-alexandre) wrote :

You need to modify:

- the manifest.json file to update the name as well,
- the Exec line for the desktop file contains a base64 encoded string corresponding to the name,

unity-webapps-runner -n 'R29vZ2xlQ2FsZW5kYXI='

which needs to be updated too,

- you can also change the X-Ubuntu- name,

review: Needs Fixing
71. By Robert Ancell

Update manifest name

72. By Robert Ancell

Update base64 encoded name in Exec line

73. By Robert Ancell

Update XB-Ubuntu-Webapps-Name in debian/control

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)

Unmerged revisions

73. By Robert Ancell

Update XB-Ubuntu-Webapps-Name in debian/control

72. By Robert Ancell

Update base64 encoded name in Exec line

71. By Robert Ancell

Update manifest name

70. By Robert Ancell

Correct name from GoogleCalendar to Google Calendar

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'GoogleCalendargooglecom.desktop'
--- GoogleCalendargooglecom.desktop 2014-04-08 20:56:01 +0000
+++ GoogleCalendargooglecom.desktop 2014-05-13 20:07:48 +0000
@@ -1,9 +1,9 @@
1[Desktop Entry]1[Desktop Entry]
2Name=GoogleCalendar2Name=Google Calendar
3Type=Application3Type=Application
4Icon=unity-webapps-google-calendar4Icon=unity-webapps-google-calendar
5MimeType=5MimeType=
6Actions=S0;S1;S2;S3;S4;S5;S6;S7;S8;S9;S10;6Actions=S0;S1;S2;S3;S4;S5;S6;S7;S8;S9;S10;
7Exec=unity-webapps-runner -n 'R29vZ2xlQ2FsZW5kYXI=' -d 'google.com' --store-session-cookies %u7Exec=unity-webapps-runner -n 'R29vZ2xlIENhbGVuZGFy' -d 'google.com' --store-session-cookies %u
8StartupWMClass=GoogleCalendargooglecom8StartupWMClass=GoogleCalendargooglecom
9 9
10\ No newline at end of file10\ No newline at end of file
1111
=== modified file 'debian/control'
--- debian/control 2013-09-13 21:20:50 +0000
+++ debian/control 2014-05-13 20:07:48 +0000
@@ -17,7 +17,7 @@
17 xdg-utils,17 xdg-utils,
18 ${misc:Depends},18 ${misc:Depends},
19XB-Ubuntu-Webapps-Includes: https://www.google.*/calendar/*;http://www.google.*/calendar/*19XB-Ubuntu-Webapps-Includes: https://www.google.*/calendar/*;http://www.google.*/calendar/*
20XB-Ubuntu-Webapps-Name: GoogleCalendar20XB-Ubuntu-Webapps-Name: Google Calendar
21XB-Ubuntu-Webapps-Domain: google.com21XB-Ubuntu-Webapps-Domain: google.com
22Description: Unity Webapp for GoogleCalendar22Description: Unity Webapp for Google Calendar
23 GoogleCalendar Webapp for Unity23 Google Calendar Webapp for Unity
2424
=== modified file 'manifest.json'
--- manifest.json 2014-04-08 20:56:01 +0000
+++ manifest.json 2014-05-13 20:07:48 +0000
@@ -1,1 +1,1 @@
1{"includes":["https://www.google.*/calendar/*","http://www.google.*/calendar/*","https://accounts.google.*/*", "https://www.google.*/a/*"],"requires":["utils.js"],"name":"GoogleCalendar","scripts":["GoogleCalendar.user.js"],"maintainer":"Webapps Team <webapps@lists.launchpad.net>","manifest-version":"1.0","integration-version":"2.2","package-name":"GoogleCalendar","icons":{"128":"128/unity-webapps-google-calendar.png","48":"48/unity-webapps-google-calendar.png","52":"52/unity-webapps-google-calendar.png","64":"64/unity-webapps-google-calendar.png"},"domain":"google.com","homepage":"https://www.google.com/calendar/render","license":"GPL-3"}1{"includes":["https://www.google.*/calendar/*","http://www.google.*/calendar/*","https://accounts.google.*/*", "https://www.google.*/a/*"],"requires":["utils.js"],"name":"Google Calendar","scripts":["GoogleCalendar.user.js"],"maintainer":"Webapps Team <webapps@lists.launchpad.net>","manifest-version":"1.0","integration-version":"2.2","package-name":"GoogleCalendar","icons":{"128":"128/unity-webapps-google-calendar.png","48":"48/unity-webapps-google-calendar.png","52":"52/unity-webapps-google-calendar.png","64":"64/unity-webapps-google-calendar.png"},"domain":"google.com","homepage":"https://www.google.com/calendar/render","license":"GPL-3"}

Subscribers

People subscribed via source and target branches

to all changes: