Let Firefox Account avatars ride the trains in Fennec

VERIFIED FIXED in Firefox 42

Status

()

VERIFIED FIXED
3 years ago
a year ago

People

(Reporter: nalexander, Assigned: nalexander)

Tracking

(Blocks: 1 bug, {user-doc-needed})

unspecified
Firefox 43
user-doc-needed
Points:
---

Firefox Tracking Flags

(firefox42+ verified, firefox43 verified)

Details

Attachments

(1 attachment)

Avatars are behind a build flag, preffed on only in Nightly for now.  This ticket tracks flipping the switch and letting them ride the trains.
(Assignee)

Comment 1

3 years ago
Edwin, Kevin: can I get some focused testing for this feature?  I think the following tests are sufficient.

1) create Firefox Account on Desktop; configure avatar image.  (There's UX for this in Desktop Nightly -- go to the Sync preferences and click the avatar image.)
2) install recent Fennec Nightly;
3) connect Fennec to Firefox Account;
4) go to Fennec > Settings > Sync and force a sync;
5) observe avatar image downloaded and displayed.

As a separate test:

6) kill and re-open Fennec;
7) go to Fennec > Settings > Sync;
8) observe avatar image downloaded and displayed.  (May flicker if it re-downloads.)

Now:

9) return to Desktop; change avatar image to a new photo.
10) return to Fennec;
11) go to Fennec > Settings > Sync, force Sync;
12) observe updated avatar image downloaded and displayed.
Flags: needinfo?(kbrosnan)
Flags: needinfo?(edwong)

Comment 2

3 years ago
Peter - do you have any cycles for this - I'll investigate why the avatar isn't showing on my install of nightly.
Flags: needinfo?(edwong) → needinfo?(pdehaan)
(Assignee)

Comment 3

3 years ago
Created attachment 8645829 [details]
MozReview Request: Bug 1192004 - Enable Firefox Account avatars in Fennec release builds. r?margaret

Bug 1192004 - Enable Firefox Account avatars in Fennec release builds. r?margaret
Attachment #8645829 - Flags: review?(margaret.leibovic)
(Assignee)

Comment 4

3 years ago
I got confirmation this was working in #fxa, from Edwin and another tester.  Leaving the flag for kbrosnan so he's aware of this new Fennec functionality.
Flags: needinfo?(pdehaan)
(Assignee)

Comment 5

3 years ago
[Tracking Requested - why for this release]: Desktop parity.
tracking-firefox42: --- → ?

Comment 6

3 years ago
Comment on attachment 8645829 [details]
MozReview Request: Bug 1192004 - Enable Firefox Account avatars in Fennec release builds. r?margaret

https://reviewboard.mozilla.org/r/15551/#review14035

Ship It!
Attachment #8645829 - Flags: review?(margaret.leibovic) → review+

Comment 7

3 years ago
bbermes, we should probably make sure this is tracked in the product roadmap.
Flags: needinfo?(bbermes)
margaret, this is going out for 42? 

I'm thinking about the dates and all the deadlines for 42.

I just chatted with kar, if there is a concrete driver, and there aren't any string changes + it's easy to uplift to 42 aurora, we could do it for 42.
Flags: needinfo?(bbermes)
Flags: needinfo?(margaret.leibovic)
Flags: needinfo?(nalexander)
(Assignee)

Comment 9

3 years ago
(In reply to Barbara Bermes [:bbermes] from comment #8)
> margaret, this is going out for 42? 
> 
> I'm thinking about the dates and all the deadlines for 42.
> 
> I just chatted with kar, if there is a concrete driver, and there aren't any
> string changes + it's easy to uplift to 42 aurora, we could do it for 42.

Yes, this should track 42.  It's just flipping the flag, which I should have done earlier.
Flags: needinfo?(nalexander)
Flags: needinfo?(margaret.leibovic)
(Assignee)

Comment 10

3 years ago
url:        https://hg.mozilla.org/integration/fx-team/rev/42037547a8b9b86daaa5a679232930d2c48e168a
changeset:  42037547a8b9b86daaa5a679232930d2c48e168a
user:       Nick Alexander <nalexander@mozilla.com>
date:       Mon Aug 10 11:07:29 2015 -0700
description:
Bug 1192004 - Enable Firefox Account avatars in Fennec release builds. r=margaret
(Assignee)

Comment 11

3 years ago
Comment on attachment 8645829 [details]
MozReview Request: Bug 1192004 - Enable Firefox Account avatars in Fennec release builds. r?margaret

Approval Request Comment
[Feature/regressing bug #]: Firefox Account avatar support in Fennec past Nightly.
[User impact if declined]: no avatars!  Fennec will lag Desktop.
[Describe test coverage new/current, TreeHerder]: manual.
[Risks and why]: very low.  This has been on Nightly for most of a cycle. In the failure case, users see the default avatar image.
[String/UUID change made/needed]: none -- everything has already landed and been uplifted.  This is just flipping the flag to enable the feature on Aurora and above.
Attachment #8645829 - Flags: approval-mozilla-aurora?
done, added card and put it in 42 Aha column - thanks - sounds good.
https://hg.mozilla.org/mozilla-central/rev/42037547a8b9
Status: ASSIGNED → RESOLVED
Last Resolved: 3 years ago
status-firefox43: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 43
Release Note Request (optional, but appreciated)
[Why is this notable]: We have this in Desktop in 41, now in fennec, in 42.
[Suggested wording]: Ability to set a profile picture for your Firefox Account
[Links (documentation, blog post, etc)]:
status-firefox42: --- → affected
tracking-firefox42: ? → +
relnote-firefox: --- → ?
Comment on attachment 8645829 [details]
MozReview Request: Bug 1192004 - Enable Firefox Account avatars in Fennec release builds. r?margaret

Sure!
Attachment #8645829 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Using the steps from comment #1 I'm verifying this as fixed on latest Nightly/Aurora
Tested on N7 (5.1.1) and Note 3 (4.4.2)
Status: RESOLVED → VERIFIED
status-firefox42: fixed → verified
status-firefox43: fixed → verified
Keywords: user-doc-needed
Added to the release notes with "Ability to set a profile picture for a Firefox Account" as wording.
relnote-firefox: ? → 42+
(Assignee)

Comment 19

3 years ago
This no longer requires kbrosnan's attention.

sylvestre: unfortunately, that's not good working for Fennec.  (It is good wording for desktop.)  On Fennec we don't have an affordance to /set/ the profile picture; we have support for displaying existing profile pictures.  How about:

"View profile pictures associated to Firefox Account in throughout Firefox for Android"

or similar?
Flags: needinfo?(kbrosnan) → needinfo?(sledru)

Comment 20

3 years ago
(In reply to Nick Alexander :nalexander from comment #19)
> This no longer requires kbrosnan's attention.
> 
> sylvestre: unfortunately, that's not good working for Fennec.  (It is good
> wording for desktop.)  On Fennec we don't have an affordance to /set/ the
> profile picture; we have support for displaying existing profile pictures. 
> How about:
> 
> "View profile pictures associated to Firefox Account in throughout Firefox
> for Android"
> 
> or similar?

Notable as a release note? 

Sounded much cooler when the user could actually do something.
As Fabio suggested, I removed it. It is not super interesting for now...
Flags: needinfo?(sledru)
relnote-firefox: 42+ → ---

Updated

a year ago
Product: Android Background Services → Firefox for Android
You need to log in before you can comment on or make changes to this bug.