Merge lp:~mardy/ubuntu-system-settings-online-accounts/add-another into lp:~online-accounts/ubuntu-system-settings-online-accounts/master

Proposed by Alberto Mardegan
Status: Merged
Merged at revision: 197
Proposed branch: lp:~mardy/ubuntu-system-settings-online-accounts/add-another
Merge into: lp:~online-accounts/ubuntu-system-settings-online-accounts/master
Diff against target: 20 lines (+2/-2)
1 file modified
online-accounts-ui/qml/ProviderRequest.qml (+2/-2)
To merge this branch: bzr merge lp:~mardy/ubuntu-system-settings-online-accounts/add-another
Reviewer Review Type Date Requested Status
David Barth (community) Approve
Review via email: mp+237558@code.launchpad.net

Commit message

Fix typo on component name

Also, fix the page title to show the provider name.

Description of the change

Fix typo on component name

Also, fix the page title to show the provider name.

To post a comment you must log in.
Revision history for this message
David Barth (dbarth) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'online-accounts-ui/qml/ProviderRequest.qml'
2--- online-accounts-ui/qml/ProviderRequest.qml 2014-09-25 11:40:59 +0000
3+++ online-accounts-ui/qml/ProviderRequest.qml 2014-10-08 09:44:58 +0000
4@@ -124,14 +124,14 @@
5 provider: providerInfo
6 onDenied: root.denied()
7 onAllowed: root.grantAccess(accountId)
8- onCreateAccount: pageStack.push(accountCreationPage)
9+ onCreateAccount: pageStack.push(createAccountPageComponent)
10 }
11 }
12
13 Component {
14 id: createAccountPageComponent
15 Page {
16- title: i18n.tr("Accounts")
17+ title: providerInfo.displayName
18 Loader {
19 anchors.fill: parent
20 sourceComponent: accountCreationPage

Subscribers

People subscribed via source and target branches