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

Proposed by Evan McIntire
Status: Superseded
Proposed branch: lp:~mcintire-evan/ubuntu-terminal-app/change_auth_dialog
Merge into: lp:~ubuntu-terminal-dev/ubuntu-terminal-app/reboot
Diff against target: 48 lines (+5/-5)
2 files modified
po/com.ubuntu.terminal.pot (+3/-3)
src/app/qml/AuthenticationDialog.qml (+2/-2)
To merge this branch: bzr merge lp:~mcintire-evan/ubuntu-terminal-app/change_auth_dialog
Reviewer Review Type Date Requested Status
Alan Pope 🍺🐧🐱 πŸ¦„ (community) Approve
Nicholas Skaggs (community) Needs Fixing
Jenkins Bot continuous-integration Needs Fixing
Review via email: mp+280781@code.launchpad.net

This proposal has been superseded by a proposal from 2016-01-20.

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 :

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 :

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

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

Great! Thanks Evan.

review: Approve
Revision history for this message
Jenkins Bot (ubuntu-core-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/~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 :

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 :

:-(

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 :

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

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

Works! No typos! :D

review: Approve

Unmerged revisions

144. By Evan McIntire

Fixed type ('passcore' -> 'passcode')

143. By Evan McIntire

Change Auth Dialog to be in line with other apps
("Enter Password" to "Enter Passcode or Passphrase")

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-08-09 14:03:25 +0000
+++ po/com.ubuntu.terminal.pot 2015-12-29 23:24:02 +0000
@@ -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-08-09 16:01+0200\n"11"POT-Creation-Date: 2015-12-29 18:00-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 2015-12-29 23:24:02 +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