Merge lp:~popey/reminders-app/fix-1434500 into lp:reminders-app

Status: Merged
Approved by: Michael Zanetti
Approved revision: 388
Merged at revision: 442
Proposed branch: lp:~popey/reminders-app/fix-1434500
Merge into: lp:reminders-app
Diff against target: 66 lines (+8/-8)
4 files modified
com.ubuntu.reminders.desktop.in.in (+2/-2)
com.ubuntu.reminders_reminders.application.in (+1/-1)
debian/control (+3/-3)
manifest.json.in (+2/-2)
To merge this branch: bzr merge lp:~popey/reminders-app/fix-1434500
Reviewer Review Type Date Requested Status
Michael Zanetti (community) Approve
Ubuntu Phone Apps Jenkins Bot continuous-integration Approve
Review via email: mp+253651@code.launchpad.net

Commit message

Rename user facing parts from Reminders to Notes

To post a comment you must log in.
Revision history for this message
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
Michael Zanetti (mzanetti) wrote :

I've been using the app a bit and watching out for issues with this: One of them is that the OAuth plugin requests the user to grant access to the "Reminders" app. Unfortunately, that string is located on the Evernote end and we can't change that ourselved.

Also I'm still really not sure if it's a good idea to just rename some parts, keeping appid to reminders. I'm worried his will cause confusion.

Revision history for this message
Alan Pope 🍺🐧🐱 πŸ¦„ (popey) wrote :

Further to our various discussions about this branch on IRC and hangouts, shall we land, build a click and do some wider testing?

Revision history for this message
Michael Zanetti (mzanetti) :
review: Needs Fixing
lp:~popey/reminders-app/fix-1434500 updated
388. By Alan Pope 🍺🐧🐱 πŸ¦„

Fix erroneous rename

Revision history for this message
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
Michael Zanetti (mzanetti) wrote :

yeehaa!

I propose following plan:

a) Build a package with the 3 critical fixes, the rename and the icon
b) Distribute it for testing to some known users
c) Contact Evernote to update the key's name from Reminders to Notes

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'com.ubuntu.reminders.desktop.in.in'
2--- com.ubuntu.reminders.desktop.in.in 2014-09-28 08:31:38 +0000
3+++ com.ubuntu.reminders.desktop.in.in 2015-05-28 09:49:37 +0000
4@@ -1,6 +1,6 @@
5 [Desktop Entry]
6-_Name=Reminders
7-_Comment=Ubuntu Reminders app, powered by Evernote
8+_Name=Notes
9+_Comment=Ubuntu Notes app, powered by Evernote
10 Exec=@EXEC@
11 Icon=@ICON@
12 Terminal=false
13
14=== modified file 'com.ubuntu.reminders_reminders.application.in'
15--- com.ubuntu.reminders_reminders.application.in 2014-09-24 15:02:10 +0000
16+++ com.ubuntu.reminders_reminders.application.in 2015-05-28 09:49:37 +0000
17@@ -1,7 +1,7 @@
18 <?xml version="1.0" encoding="UTF-8" ?>
19 <application id="@PROJECT_NAME@_@APP_NAME@">
20
21- <description>Reminders</description>
22+ <description>Notes</description>
23
24 <services>
25 <service id="com.ubuntu.reminders_reminders">
26
27=== modified file 'debian/control'
28--- debian/control 2014-12-16 22:35:59 +0000
29+++ debian/control 2015-05-28 09:49:37 +0000
30@@ -34,7 +34,7 @@
31 suru-icon-theme | ubuntu-mobile-icons,
32 ${misc:Depends},
33 ${shlibs:Depends}
34-Description: Reminders application
35+Description: Notes application
36 Note taking application powered by Evernote
37
38 Package: qtdeclarative5-evernote0.1
39@@ -79,5 +79,5 @@
40 reminders-app (>= ${source:Version}),
41 ubuntu-ui-toolkit-autopilot,
42 ${misc:Depends}
43-Description: Test package for the Reminders app
44- Autopilot tests for the Reminders app package
45+Description: Test package for the Notes app
46+ Autopilot tests for the Notes app package
47
48=== modified file 'manifest.json.in'
49--- manifest.json.in 2015-02-12 20:34:11 +0000
50+++ manifest.json.in 2015-05-28 09:49:37 +0000
51@@ -1,5 +1,5 @@
52 {
53- "description": "Ubuntu Reminders app, powered by Evernote",
54+ "description": "Ubuntu Notes app, powered by Evernote",
55 "framework": "ubuntu-sdk-14.10",
56 "architecture": "@CLICK_ARCH@",
57 "hooks": {
58@@ -21,7 +21,7 @@
59 },
60 "maintainer": "Ubuntu App Cats <ubuntu-touch-coreapps@lists.launchpad.net>",
61 "name": "com.ubuntu.reminders",
62- "title": "Reminders",
63+ "title": "Notes",
64 "version": "0.5.@BZR_REVNO@",
65 "x-source": {
66 "vcs-bzr": "lp:reminders-app",

Subscribers

People subscribed via source and target branches