Provide feedback to user that we can't sync passwords with MP enabled

NEW
Unassigned

Status

()

Firefox for Android
Android Sync
P3
normal
6 years ago
7 months ago

People

(Reporter: wesj, Unassigned)

Tracking

(Blocks: 1 bug)

unspecified
All
Android
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(fennec+)

Details

(Whiteboard: [qa+][parallel])

(Reporter)

Description

6 years ago
Bug 757646 made me realize we don't really provide any feedback to the user about Sync failing to sync passwords when MP is enabled. We should do something... a one time toast or notification? Or a description line in the settings activity?

Updated

6 years ago
tracking-fennec: --- → ?
We're working on a settings dialog to control what to sync. I'd suggest two places to surface this: one being that Sync settings dialog, and the other being a description line underneath the Master Password checkbox -- "Enabling Master Password will prevent Firefox Sync from synchronizing your passwords.".

That way a user is bound to see something in their path, without the annoyance of a toast.

Thoughts?
OS: Linux → Android
Hardware: x86 → ARM
tracking-fennec: ? → 19+
tracking-fennec: 19+ → +

Comment 2

4 years ago
https://support.mozilla.org/en-US/questions/974139 ("Does Firefox for Android sync passwords from Firefox desktop?") is related.
(Btw., count me as another one hit by this bug.)
Blocks: 951295
Hardware: ARM → All
Summary: Provide feedback to use the we can't sync passwords with MP enabled → Provide feedback to user that we can't sync passwords with MP enabled
Component: General → Android Sync
Product: Firefox for Android → Android Background Services
Version: Trunk → unspecified
Whiteboard: [qa+]
Blocks: 799726
Flags: needinfo?(deb)
We do have to let the user know what's going on, but I'm going to let ibarlow sort out the how/where part.  So yes, we need this, but NEEDINFOing ibarlow for design-related decisions.

Eventually this should be moot, but that day is not upon us :)
Flags: needinfo?(deb) → needinfo?(ibarlow)
Some relevant future context: Bug 973759 Comment 6, and the rest of the thread.

Deb, do you have a target release for this? If it's later than 31, we should probably just consider doing what I propose in that comment.
Whiteboard: [qa+] → [qa+][parallel]

Comment 5

4 years ago
triage: polish bug
Priority: -- → P3
(In reply to Deb Richardson [:dria] from comment #3)
> We do have to let the user know what's going on, but I'm going to let
> ibarlow sort out the how/where part.  So yes, we need this, but NEEDINFOing
> ibarlow for design-related decisions.
> 
> Eventually this should be moot, but that day is not upon us :)

I'd rather we try to fix master password or remove it altogether first, instead of putting this little bandaid on the problem. 

yes, a note about "enabling this will make it so your passwords don't sync" would explain that small slice of Master Password to users, but it doesn't solve the bigger problem of master password not actually functioning as a feature in the way anyone expects it to, or that we don't actually explain it to them anywhere.
Flags: needinfo?(ibarlow)

Updated

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