Merge lp:~mcintire-evan/ubuntu-terminal-app/change-auth into lp:~ubuntu-terminal-dev/ubuntu-terminal-app/reboot

Proposed by Evan McIntire
Status: Merged
Approved by: Niklas Wenzel
Approved revision: 152
Merged at revision: 152
Proposed branch: lp:~mcintire-evan/ubuntu-terminal-app/change-auth
Merge into: lp:~ubuntu-terminal-dev/ubuntu-terminal-app/reboot
Diff against target: 56 lines (+6/-6)
2 files modified
po/com.ubuntu.terminal.pot (+4/-4)
src/app/qml/AuthenticationDialog.qml (+2/-2)
To merge this branch: bzr merge lp:~mcintire-evan/ubuntu-terminal-app/change-auth
Reviewer Review Type Date Requested Status
Jenkins Bot continuous-integration Approve
Niklas Wenzel (community) Approve
Alan Pope 🍺🐧🐱 πŸ¦„ (community) Approve
Nicholas Skaggs Pending
Review via email: mp+283240@code.launchpad.net

This proposal supersedes a proposal from 2015-12-16.

Commit message

Changes authentication dialogue to "Enter passcode or passphrase" instead of "Enter password", to be more in line with other parts of the OS.

Description of the change

Changes authentication dialogue to "Enter passcode or passphrase" instead of "Enter password", to be more in line with other parts of the OS

To post a comment you must log in.
Revision history for this message
Niklas Wenzel (nikwen) wrote : Posted in a previous version of this proposal

Thank you for your contribution!

Please see my inline comment. If you fix it, we can happily merge this. ;)

Revision history for this message
Evan McIntire (mcintire-evan) wrote : Posted in a previous version of this proposal

Whoops, can't believe I made that mistake. Uploading a fix in a moment, thanks! :)

Revision history for this message
Alan Pope 🍺🐧🐱 πŸ¦„ (popey) wrote : Posted in a previous version of this proposal

Great! Thanks Evan.

review: Approve
Revision history for this message
Jenkins Bot (ubuntu-core-apps-jenkins-bot) wrote : Posted in a previous version of this proposal

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/~mcintire-evan/ubuntu-terminal-app/change_auth_dialog/+merge/280781/+edit-commit-message

review: Needs Fixing (continuous-integration)
Revision history for this message
Jenkins Bot (ubuntu-core-apps-jenkins-bot) wrote : Posted in a previous version of this proposal

FAILED: Autolanding.
More details in the following jenkins job:
https://core-apps-jenkins.ubuntu.com/job/terminal-app-autolanding/9/
Executed test runs:
    None: https://core-apps-jenkins.ubuntu.com/job/generic-land-mp/1461/console

review: Needs Fixing (continuous-integration)
Revision history for this message
Nicholas Skaggs (nskaggs) wrote : Posted in a previous version of this proposal

:-(

Text conflict in po/com.ubuntu.terminal.pot
1 conflicts encountered.
bzr: ERROR: Conflicts from merge

review: Needs Fixing
Revision history for this message
Nicholas Skaggs (nskaggs) wrote : Posted in a previous version of this proposal

This needs to land after updating so pot file doesn't cause havoc again.

Revision history for this message
Evan McIntire (mcintire-evan) wrote :

Redid it with a fresh branch, and no typo this time :)

Revision history for this message
Alan Pope 🍺🐧🐱 πŸ¦„ (popey) wrote : Posted in a previous version of this proposal

Works! No typos! :D

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

I suspect I left a comment in the wrong place :)

review: Approve
Revision history for this message
Niklas Wenzel (nikwen) wrote :

I think we need to kick off the landing here. Thank you for the patch and for fixing the typo. :)

(Sadly, I didn't get any notification that you updated the MP so that's why I didn't respond earlier ...)

review: Approve
Revision history for this message
Jenkins Bot (ubuntu-core-apps-jenkins-bot) :
review: Approve (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'po/com.ubuntu.terminal.pot'
--- po/com.ubuntu.terminal.pot 2015-12-19 05:14:39 +0000
+++ po/com.ubuntu.terminal.pot 2016-01-20 01:26:16 +0000
@@ -1,6 +1,6 @@
1# SOME DESCRIPTIVE TITLE.1# SOME DESCRIPTIVE TITLE.
2# Copyright (C) YEAR Canonical Ltd.2# Copyright (C) YEAR Canonical Ltd.
3# This file is distributed under the same license as the PACKAGE package.3# This file is distributed under the same license as the package.
4# FIRST AUTHOR <EMAIL@ADDRESS>, YEAR.4# FIRST AUTHOR <EMAIL@ADDRESS>, YEAR.
5#5#
6#, fuzzy6#, fuzzy
@@ -8,7 +8,7 @@
8msgstr ""8msgstr ""
9"Project-Id-Version: \n"9"Project-Id-Version: \n"
10"Report-Msgid-Bugs-To: \n"10"Report-Msgid-Bugs-To: \n"
11"POT-Creation-Date: 2015-12-19 00:08-0500\n"11"POT-Creation-Date: 2016-01-19 20:23-0500\n"
12"PO-Revision-Date: YEAR-MO-DA HO:MI+ZONE\n"12"PO-Revision-Date: YEAR-MO-DA HO:MI+ZONE\n"
13"Last-Translator: FULL NAME <EMAIL@ADDRESS>\n"13"Last-Translator: FULL NAME <EMAIL@ADDRESS>\n"
14"Language-Team: LANGUAGE <LL@li.org>\n"14"Language-Team: LANGUAGE <LL@li.org>\n"
@@ -34,11 +34,11 @@
34msgstr ""34msgstr ""
3535
36#: ../src/app/qml/AuthenticationDialog.qml:2736#: ../src/app/qml/AuthenticationDialog.qml:27
37msgid "Enter password:"37msgid "Enter passcode or passphrase:"
38msgstr ""38msgstr ""
3939
40#: ../src/app/qml/AuthenticationDialog.qml:4040#: ../src/app/qml/AuthenticationDialog.qml:40
41msgid "password"41msgid "passcode or passphrase"
42msgstr ""42msgstr ""
4343
44#: ../src/app/qml/AuthenticationDialog.qml:5044#: ../src/app/qml/AuthenticationDialog.qml:50
4545
=== modified file 'src/app/qml/AuthenticationDialog.qml'
--- src/app/qml/AuthenticationDialog.qml 2015-07-13 20:25:21 +0000
+++ src/app/qml/AuthenticationDialog.qml 2016-01-20 01:26:16 +0000
@@ -24,7 +24,7 @@
2424
25 title: i18n.tr("Authentication required")25 title: i18n.tr("Authentication required")
2626
27 text: i18n.tr("Enter password:")27 text: i18n.tr("Enter passcode or passphrase:")
2828
29 signal passwordEntered(string password)29 signal passwordEntered(string password)
30 signal dialogCanceled30 signal dialogCanceled
@@ -37,7 +37,7 @@
37 id: passwordField37 id: passwordField
38 objectName: "inputField"38 objectName: "inputField"
3939
40 placeholderText: i18n.tr("password")40 placeholderText: i18n.tr("passcode or passphrase")
41 echoMode: TextInput.Password41 echoMode: TextInput.Password
4242
43 onAccepted: okButton.clicked(text)43 onAccepted: okButton.clicked(text)

Subscribers

People subscribed via source and target branches