Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Promote password import in autofill menu #3220

Merged
merged 32 commits into from
Sep 10, 2024

Conversation

graeme
Copy link
Collaborator

@graeme graeme commented Sep 5, 2024

Task/Issue URL: https://app.asana.com/0/72649045549333/1207617229892922/f
BSK PR: duckduckgo/BrowserServicesKit#976

Description:

When adopting a new browser after a long time using something else, you expect your browser to have all your data. If you skipped import during onboarding you will find out about the missing data at the time when you're trying to use the password. This creates huge friction right in the middle of a task. See Comment by Peter Dolanjski on Password and autofill issues.
Reminding users of the option to import right when that friction happens could be a way to quickly give recourse and solve the problem for all subsequent browsing needs.

This feature promotes the import in autofill scenarios when the user has few or no passwords saved.

Steps to test this PR:
https://app.asana.com/0/72649045549333/1208245520043990/f

Definition of Done:


Internal references:

Pull Request Review Checklist
Software Engineering Expectations
Technical Design Template
Pull Request Documentation

@graeme graeme requested a review from amddg44 September 6, 2024 13:24
@graeme graeme marked this pull request as ready for review September 6, 2024 13:24
Copy link
Contributor

@amddg44 amddg44 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Just one small edge case I noticed - when i imported passwords while on a site that I didn’t have a password for, I got this empty overlay at the end of the import flow
image

@dbajpeyi
Copy link
Collaborator

dbajpeyi commented Sep 10, 2024

@amddg44 Just for posterity the bug you mentioned was already fixed in autofill.js and is in the 13.1.0 version (Release Candidate)
duckduckgo/duckduckgo-autofill#649

So we should be already good with the tagged version :)

graeme added a commit to duckduckgo/BrowserServicesKit that referenced this pull request Sep 10, 2024
Please review the release process for BrowserServicesKit
[here](https://app.asana.com/0/1200194497630846/1200837094583426).

**Required**:

Task/Issue URL:
https://app.asana.com/0/72649045549333/1207617229892922/f
iOS PR: duckduckgo/iOS#3332
macOS PR: duckduckgo/macos-browser#3220
What kind of version bump will this require?: Major

**Optional**:

Tech Design URL:
https://app.asana.com/0/481882893211075/1208037181667142

**Description**:

When adopting a new browser after a long time using something else, you
expect your browser to have all your data. If you skipped import during
onboarding you will find out about the missing data at the time when
you're trying to use the password. This creates huge friction right in
the middle of a task. See [Comment by Peter Dolanjski on Password and
autofill
issues](https://app.asana.com/0/0/1207533847538452/1207544369978642/f).
Reminding users of the option to import right when that friction happens
could be a way to quickly give recourse and solve the problem for all
subsequent browsing needs.

This feature promotes the import in autofill scenarios when the user has
few or no passwords saved.

**Steps to test this PR**:
(Same as macOS PR)
https://app.asana.com/0/72649045549333/1208245520043990/f

**OS Testing**:

* [ ] iOS 14
* [ ] iOS 15
* [ ] iOS 16
* [ ] macOS 10.15
* [ ] macOS 11
* [ ] macOS 12

---
###### Internal references:
[Software Engineering
Expectations](https://app.asana.com/0/59792373528535/199064865822552)
[Technical Design
Template](https://app.asana.com/0/59792373528535/184709971311943)
@graeme graeme merged commit ba40958 into main Sep 10, 2024
18 checks passed
@graeme graeme deleted the graeme/promote-password-import-in-autofill-menu branch September 10, 2024 12:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants