Merge lp://qastaging/~renatofilho/address-book-app/update-avatar-when-name-change into lp://qastaging/~phablet-team/address-book-app/staging
Proposed by
Renato Araujo Oliveira Filho
Status: | Merged |
---|---|
Approved by: | Arthur Mello |
Approved revision: | 239 |
Merged at revision: | 239 |
Proposed branch: | lp://qastaging/~renatofilho/address-book-app/update-avatar-when-name-change |
Merge into: | lp://qastaging/~phablet-team/address-book-app/staging |
Diff against target: |
74 lines (+7/-30) 2 files modified
src/imports/ContactView/ContactDetailAvatarView.qml (+2/-28) src/imports/Ubuntu/Contacts/ContactAvatar.qml (+5/-2) |
To merge this branch: | bzr merge lp://qastaging/~renatofilho/address-book-app/update-avatar-when-name-change |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Arthur Mello (community) | Approve | ||
PS Jenkins bot | continuous-integration | Approve | |
Review via email:
|
Commit message
Fix Avatar reload function to update contact display name and avatar url.
This is necessary for contact view in case of the contact name is changed during the edit mode.
Description of the change
How to test
===========
1 Create a new contact set a avatar picture check it was saved correct
2 Edit a contact without avatar
2.1 - Change contact name and save (check in contact view if the avatar got updated to the correct initials)
2.2 - Set a contact avatar check if the contact avatar get update
To post a comment you must log in.
PASSED: Continuous integration, rev:239 jenkins. qa.ubuntu. com/job/ phablet- team-address- book-app- staging- ci/278/ jenkins. qa.ubuntu. com/job/ generic- deb-autopilot- utopic- touch/3395 jenkins. qa.ubuntu. com/job/ generic- mediumtests- utopic/ 2666 jenkins. qa.ubuntu. com/job/ phablet- team-address- book-app- staging- utopic- amd64-ci/ 278 jenkins. qa.ubuntu. com/job/ phablet- team-address- book-app- staging- utopic- armhf-ci/ 278 jenkins. qa.ubuntu. com/job/ phablet- team-address- book-app- staging- utopic- armhf-ci/ 278/artifact/ work/output/ *zip*/output. zip jenkins. qa.ubuntu. com/job/ phablet- team-address- book-app- staging- utopic- i386-ci/ 278 jenkins. qa.ubuntu. com/job/ generic- deb-autopilot- runner- mako/3320 jenkins. qa.ubuntu. com/job/ generic- mediumtests- builder- utopic- armhf/4642 jenkins. qa.ubuntu. com/job/ generic- mediumtests- builder- utopic- armhf/4642/ artifact/ work/output/ *zip*/output. zip s-jenkins. ubuntu- ci:8080/ job/touch- flash-device/ 11321 jenkins. qa.ubuntu. com/job/ autopilot- testrunner- otto-utopic/ 2156 jenkins. qa.ubuntu. com/job/ generic- mediumtests- builder- utopic- amd64/2943 jenkins. qa.ubuntu. com/job/ generic- mediumtests- builder- utopic- amd64/2943/ artifact/ work/output/ *zip*/output. zip
http://
Executed test runs:
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
deb: http://
Click here to trigger a rebuild: s-jenkins. ubuntu- ci:8080/ job/phablet- team-address- book-app- staging- ci/278/ rebuild
http://