Merge lp:~arashbm/ubuntu-filemanager-app/translation-fix into lp:ubuntu-filemanager-app

Proposed by Arash Badie Modiri
Status: Rejected
Rejected by: David Planella
Proposed branch: lp:~arashbm/ubuntu-filemanager-app/translation-fix
Merge into: lp:ubuntu-filemanager-app
Diff against target: 45 lines (+7/-6)
2 files modified
FolderListView.qml (+1/-1)
po/ubuntu-filemanager-app.pot (+6/-5)
To merge this branch: bzr merge lp:~arashbm/ubuntu-filemanager-app/translation-fix
Reviewer Review Type Date Requested Status
David Planella Disapprove
Ubuntu Phone Apps Jenkins Bot continuous-integration Needs Fixing
Arto Jalkanen Approve
Adolfo Jayme Barrientos (community) Approve
Review via email: mp+201691@code.launchpad.net

Commit message

Translation fix

Description of the change

generalized translation string to support languages that don't have nouns at the
end of sentences.

To post a comment you must log in.
Revision history for this message
Adolfo Jayme Barrientos (fitojb) wrote :

Awesome!

review: Approve
Revision history for this message
Arto Jalkanen (ajalkane) wrote :

Thank you

review: Approve
Revision history for this message
David Planella (dpm) wrote :

Looks good to me, sorry for the delay, and approving!

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

FAILED: Autolanding.
No commit message was specified in the merge proposal. Hit 'Add commit message' on the merge proposal web page or follow the link below. You can approve the merge proposal yourself to rerun.
https://code.launchpad.net/~arashbm/ubuntu-filemanager-app/translation-fix/+merge/201691/+edit-commit-message

review: Needs Fixing (continuous-integration)
Revision history for this message
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
David Planella (dpm) wrote :

Hi Arash,

It looks like your branch and trunk have diverged in the meantime, would you mind doing the following, so that we can merge your branch?:

bzr branch lp:~arashbm/ubuntu-filemanager-app/translation-fix
cd translation-fix
bzr merge lp:ubuntu-filemanager-app
(fix any conflicts you encounter)
bzr commit -m"Merged from trunk" # Or any other commit message you want
bzr push lp:~arashbm/ubuntu-filemanager-app/translation-fix

This should automatically update the merge proposal in Launchpad

review: Needs Fixing
Revision history for this message
David Planella (dpm) wrote :

Hi Arash, in order to get this landed, I added your fix in a new branch, including authorship for the commit, so that you're properly credited.

Thus I'm marking this branch as rejected, since it's superseded by this newer one, which includes your contribution:
https://code.launchpad.net/~dpm/ubuntu-filemanager-app/translation-fix/+merge/217725

review: Disapprove

Unmerged revisions

111. By Arash Badie Modiri

Updated pot file

110. By Arash Badie Modiri

generalizing translation string to add persian (among other languages) support

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'FolderListView.qml'
2--- FolderListView.qml 2013-09-17 16:40:38 +0000
3+++ FolderListView.qml 2014-01-14 23:21:35 +0000
4@@ -173,7 +173,7 @@
5 PopupUtils.open(Qt.resolvedUrl("NotifyDialog.qml"), delegate,
6 {
7 title: i18n.tr("Folder not accessible"),
8- text: i18n.tr("Can not access ") + model.fileName
9+ text: i18n.tr("Can not access %1").arg(model.fileName)
10
11 })
12 }
13
14=== modified file 'po/ubuntu-filemanager-app.pot'
15--- po/ubuntu-filemanager-app.pot 2013-12-30 20:31:12 +0000
16+++ po/ubuntu-filemanager-app.pot 2014-01-14 23:21:35 +0000
17@@ -1,6 +1,6 @@
18-# Filemanager App Translations
19-# Copyright (C) 2013 Canonical Ltd.
20-# This file is distributed under the same license as the ubuntu-filemanager-app package.
21+# SOME DESCRIPTIVE TITLE.
22+# Copyright (C) YEAR Canonical Ltd.
23+# This file is distributed under the same license as the PACKAGE package.
24 # FIRST AUTHOR <EMAIL@ADDRESS>, YEAR.
25 #
26 #, fuzzy
27@@ -8,7 +8,7 @@
28 msgstr ""
29 "Project-Id-Version: ubuntu-filemanager-app\n"
30 "Report-Msgid-Bugs-To: \n"
31-"POT-Creation-Date: 2013-12-30 14:24-0600\n"
32+"POT-Creation-Date: 2014-01-15 02:50+0330\n"
33 "PO-Revision-Date: YEAR-MO-DA HO:MI+ZONE\n"
34 "Last-Translator: FULL NAME <EMAIL@ADDRESS>\n"
35 "Language-Team: LANGUAGE <LL@li.org>\n"
36@@ -252,7 +252,8 @@
37 msgstr ""
38
39 #: ../FolderListView.qml:176
40-msgid "Can not access "
41+#, qt-format
42+msgid "Can not access %1"
43 msgstr ""
44
45 #: ../GoToDialog.qml:25

Subscribers

People subscribed via source and target branches