1962614 - Show biometric auth CFR to users when opening the tabs tray for the first time

Status: RESOLVED FIXED

(Firefox for Android :: Privacy, task)

Description

[:Gela]

Product has confirmed we only want to show this CFR once when they first open the tabs tray.

Figma: https://www.figma.com/design/qiiSyigCsqDx9kNyxcGEev/BioAuth?node-id=307-9414&m=dev

Comment 1

[:Gela]

Attachment: WIP: Bug 1962614 - Show biometric auth CFR to users when opening the tabs tray for the first time

Comment 3

[:Gela]

Attachment: Bug 1962614 - Add strings for PBM locked CFR

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/523f992b14e8
Add strings for PBM locked CFR r=android-reviewers,matt-tighe

Comment 5

[Cosmin Sabou [:CosminS]]

Backed out changeset 523f992b14e8 (Bug 1962614) for causing fenix lint failures.

Push with failures

Failure log

Backout link

Comment 6

[Pulsebot]

Backout by [e-mail]:
https://hg.mozilla.org/mozilla-central/rev/d5d75dde90f3
Backed out changeset 523f992b14e8 for causing fenix lint failures. CLOSED TREE

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/cf17c23e4130
Add strings for PBM locked CFR r=android-reviewers,android-l10n-reviewers,matt-tighe,delphine

Comment 8

[Norisz Fay [:noriszfay]]

https://hg.mozilla.org/mozilla-central/rev/cf17c23e4130

Comment 9

[:Gela]

Attachment: Bug 1962614 - Add strings for PBM locked CFR

Original Revision: https://phabricator.services.mozilla.com/D246787

Comment 10

[Phabricator Automation]

beta Uplift Approval Request

• User impact if declined: Missing strings required for biometric auth prompt
• Code covered by automated testing: yes
• Fix verified in Nightly: no
• Needs manual QE test: no
• Steps to reproduce for manual QE testing: N/A. Strings only
• Risk associated with taking this patch: N/A
• Explanation of risk level: Low, just adding strings.
• String changes made/needed: Yes
• Is Android affected?: yes

Comment 11

[Ryan VanderMeulen [:RyanVM]]

Hi Bryan, any concerns with uplifting these new strings to Beta?

Comment 12

[:Gela]

Attachment: Bug 1962614 - Add banner to encourage users to enable biometric authentication

Comment 13

[Francesco Lodolo [:flod]]

(this is not desktop, redirecting)

Comment 14

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/c38333bb0f2c
Add banner to encourage users to enable biometric authentication r=android-reviewers,twhite,android-l10n-reviewers,delphine

Comment 15

[Delphine Lebédel]

String freeze is past so there is chance some locales may not get to localize these strings in time - which may lead to poor UI experince. Since there are not that many strings and if they are crucial to this release, then let's uplift them

Comment 16

[Pulsebot]

https://hg.mozilla.org/releases/mozilla-beta/rev/e3bf82104aca

Comment 17

[Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)]

https://hg.mozilla.org/mozilla-central/rev/c38333bb0f2c