Skip to content
This repository has been archived by the owner on Feb 20, 2023. It is now read-only.

AC API for integrating GV Login Autofill Delegate with Logins Storage #5543

Closed
3 of 5 tasks
ekager opened this issue Sep 24, 2019 · 6 comments
Closed
3 of 5 tasks

AC API for integrating GV Login Autofill Delegate with Logins Storage #5543

ekager opened this issue Sep 24, 2019 · 6 comments
Assignees
Labels
E5 Estimation Point: about 5 days eng:ready Ready for engineering Feature:Logins needs:gv GeckoView bug required to fix the issue. See bugzilla.mozilla.org

Comments

@ekager
Copy link
Contributor

ekager commented Sep 24, 2019

Why/User Benefit/User Problem

When a user goes to a site with a login, it can be cumbersome to login on mobile. GV has provided the interface for the GV APIs to start working against.

What/Requirements

In AC (will be done by Fenix team):

Acceptance Criteria (how do I know when I’m done?)

  • AC wrapper around GV Login APIs, while GV is working on finishing implementation

Dependencies

No AS work bc we can already store/access logins.

Not part of this ticket:

Risks

** Needs lots of QA to try out different sites, look at user-reported issues

  • The APIs requested of GV might not cover all the login field detection and autofill, and we won't know until we try it out
    ** GV uses the same detection as Desktop, so in theory should be good enough. But we won't know until we try it out.
    ** Desktop heuristics might not work well on Mobile pages - mitigation to ask Desktop/MattN to update heuristics
  • GV might have to also support Seamless Lockwise integration in Fenix for existing Lockwise users #550 login experience for existing Lockwise users, which might have bad behavior of GV interaction between Fenix and LW login autofilling
    ** Need to talk to UX and get some UX guidance
    ** Technical limitations: GV will only allow one or the other. If we need to support both, we need to get back to GV to make changes
  • Creating a component from scratch to android-components quality

┆Issue is synchronized with this Jira Task

@ekager ekager added feature request 🌟 New functionality and improvements Feature:Logins labels Sep 24, 2019
@liuche liuche added this to Fenix P1 in Login Management Sep 24, 2019
@liuche liuche added P1 Current sprint needs:gv GeckoView bug required to fix the issue. See bugzilla.mozilla.org 🙅 waiting Issues that are blocked or has dependencies that are not ready labels Sep 24, 2019
@vesta0 vesta0 added must and removed feature request 🌟 New functionality and improvements P1 Current sprint labels Sep 25, 2019
@bifleming bifleming added the E13 Estimation Point: about 10 days label Sep 25, 2019
@vesta0 vesta0 removed this from Must in Fenix Q4 Feature Backlog Oct 3, 2019
@vesta0 vesta0 removed this from Must in Fenix Q4 Feature Backlog Oct 17, 2019
@vesta0 vesta0 added this to Prioritized Feature Backlog in Fenix Sprint Kanban Oct 17, 2019
@liuche liuche removed the 🙅 waiting Issues that are blocked or has dependencies that are not ready label Oct 21, 2019
@liuche
Copy link
Contributor

liuche commented Oct 21, 2019

According to GV this is unblocked bc https://bugzilla.mozilla.org/show_bug.cgi?id=1577003 has landed on Nightly. We still will not be able to save, but we can start autofilling from existing logins.

@liuche liuche added the eng:ready Ready for engineering label Oct 21, 2019
@liuche liuche moved this from Prioritized Feature Backlog to Sprint Backlog in Fenix Sprint Kanban Oct 25, 2019
@liuche
Copy link
Contributor

liuche commented Oct 29, 2019

After a discussion w/ GV, they're going to use more Gecko Toolkit code to do more of the autofill work. That means this is no longer ready to be worked on (but the remaining work should be much less).

@liuche liuche added 🙅 waiting Issues that are blocked or has dependencies that are not ready and removed E13 Estimation Point: about 10 days eng:ready Ready for engineering labels Oct 29, 2019
@liuche liuche moved this from Sprint Backlog to Prioritized Feature Backlog in Fenix Sprint Kanban Oct 29, 2019
@cpeterso
Copy link

The GV bug has been added to GV's November sprint.

https://bugzilla.mozilla.org/show_bug.cgi?id=1577005#c13

@vesta0
Copy link
Collaborator

vesta0 commented Nov 5, 2019

We will be able to size this by Friday when we get the interface from the GV team.

@boek boek added E5 Estimation Point: about 5 days AC-by-Fenix labels Nov 19, 2019
@liuche
Copy link
Contributor

liuche commented Nov 19, 2019

I'll work w/ Emily to break this into the GV and AC parts.

@ekager ekager changed the title My browser will auto-fill the login field when I return to a site AC API for integrating upcoming GV Login Autofill Delegate with Logins Storage Nov 19, 2019
@liuche liuche added eng:ready Ready for engineering and removed 🙅 waiting Issues that are blocked or has dependencies that are not ready labels Nov 19, 2019
@liuche liuche changed the title AC API for integrating upcoming GV Login Autofill Delegate with Logins Storage AC API for integrating GV Login Autofill Delegate with Logins Storage Nov 21, 2019
@ekager ekager self-assigned this Nov 21, 2019
@ekager ekager moved this from Prioritized Feature Backlog to In Progress in Fenix Sprint Kanban Nov 22, 2019
@liuche
Copy link
Contributor

liuche commented Jan 13, 2020

@baron-severin and @ekager landed this in AC on 1/10 with https://github.com/mozilla-mobile/android-components/pull/5316/files

I'm going to close this, because this can't be QA-verified until #6696 is done to use the AC APIs.

@liuche liuche closed this as completed Jan 13, 2020
@project-bot project-bot bot moved this from In Progress to Sprint 20.1 DONE in Fenix Sprint Kanban Jan 13, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
E5 Estimation Point: about 5 days eng:ready Ready for engineering Feature:Logins needs:gv GeckoView bug required to fix the issue. See bugzilla.mozilla.org
Projects
No open projects
Development

No branches or pull requests

6 participants