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

[Autofill] Introduce password import promotion prompt in autofill.js #631

Merged
merged 5 commits into from
Sep 5, 2024

Conversation

dbajpeyi
Copy link
Collaborator

@dbajpeyi dbajpeyi commented Aug 14, 2024

Reviewer: @GioSensation
Asana: https://app.asana.com/0/481882893211075/1208037181667142/f

Description

Adding a new API startPasswordimportFlow for native apps, and extending getAvailableInputTypes to pass a new available input type called passwordImport?: boolean that would be calculated on the native side.

  1. In Forms.js, we will test if this input type is available, and if it is then we do not add the icons and it's handlers,
  2. In HTMLTooltipUIController.js we check if passwordImport is available in topContextData and we render a special kind of tooltip UI.
  3. The special kind of tooltip UI is described in ImportPasswordTooltip.js

Steps to test


@dbajpeyi dbajpeyi force-pushed the dbajpeyi/import-promotion branch 4 times, most recently from 8202505 to a5438fd Compare August 14, 2024 17:16
@dbajpeyi dbajpeyi changed the title wip: attach new tooltip type for password promotion [Autofill] attach new tooltip type for password promotion Aug 15, 2024
@dbajpeyi dbajpeyi changed the title [Autofill] attach new tooltip type for password promotion [Autofill] attach new tooltip type for password import promotion Aug 15, 2024
@dbajpeyi dbajpeyi changed the title [Autofill] attach new tooltip type for password import promotion [Autofill] Introduce password import promotion prompt in autofill.js Aug 15, 2024
@dbajpeyi dbajpeyi force-pushed the dbajpeyi/import-promotion branch 5 times, most recently from d3a6df2 to 36649c9 Compare August 17, 2024 04:59
@dbajpeyi dbajpeyi force-pushed the dbajpeyi/import-promotion branch 18 times, most recently from 9693d58 to 8bdc694 Compare August 27, 2024 11:50
@dbajpeyi dbajpeyi force-pushed the dbajpeyi/import-promotion branch 6 times, most recently from 0907ff6 to d31aa2f Compare September 3, 2024 15:32
@dbajpeyi dbajpeyi force-pushed the dbajpeyi/import-promotion branch 2 times, most recently from 3e2ae3e to 1dffe0d Compare September 4, 2024 11:18
Copy link
Member

@GioSensation GioSensation left a comment

Choose a reason for hiding this comment

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

Everything's looking pretty good. Just a few minor notes.

src/UI/styles/autofill-tooltip-styles.css Show resolved Hide resolved
src/CredentialsImport.js Outdated Show resolved Hide resolved
src/DeviceInterface/InterfacePrototype.js Outdated Show resolved Hide resolved
src/ThirdPartyProvider.js Outdated Show resolved Hide resolved
integration-test/helpers/pages/loginPage.js Outdated Show resolved Hide resolved
@dbajpeyi dbajpeyi force-pushed the dbajpeyi/import-promotion branch 4 times, most recently from 715366f to bceb519 Compare September 5, 2024 11:15
@dbajpeyi
Copy link
Collaborator Author

dbajpeyi commented Sep 5, 2024

@GioSensation

  1. addressed comments de63b74
  2. Added another commit for fixing the windows issue 81f3204
    (With some minor change here 036d9f0)

@dbajpeyi dbajpeyi merged commit 1ea2d15 into main Sep 5, 2024
1 check passed
@dbajpeyi dbajpeyi deleted the dbajpeyi/import-promotion branch September 5, 2024 11:38
CDRussell pushed a commit to duckduckgo/Android that referenced this pull request Sep 11, 2024
Task/Issue URL:
https://app.asana.com/0/1208264544627947/1208264544627947
Autofill Release:
https://github.com/duckduckgo/duckduckgo-autofill/releases/tag/13.1.0


## Description
Updates Autofill to version
[13.1.0](https://github.com/duckduckgo/duckduckgo-autofill/releases/tag/13.1.0).

### Autofill 13.1.0 release notes
## What's Changed
https://app.asana.com/0/72649045549333/1201480346793878/f
Adding password import promotion support. There are no breaking API
changes, hence minor version bump!

* Update password-related json files (2024-08-15) by @daxmobile in
duckduckgo/duckduckgo-autofill#632
* Update password-related json files (2024-08-29) by @daxmobile in
duckduckgo/duckduckgo-autofill#644
* Update password-related json files (2024-08-30) by @daxmobile in
duckduckgo/duckduckgo-autofill#645
* Update password-related json files (2024-09-03) by @daxmobile in
duckduckgo/duckduckgo-autofill#646
* [Autofill] Introduce password import promotion prompt in autofill.js
by @dbajpeyi in
duckduckgo/duckduckgo-autofill#631
* [Locales] Credentials import by @dbajpeyi in
duckduckgo/duckduckgo-autofill#647
* [Credentials Import] remove API response by @dbajpeyi in
duckduckgo/duckduckgo-autofill#648
* [Credentials Import] Check local credentials before showing prompt by
@dbajpeyi in duckduckgo/duckduckgo-autofill#650
* [Credentials import] If no details are available don't show the
tooltip by @dbajpeyi in
duckduckgo/duckduckgo-autofill#649


**Full Changelog**:
duckduckgo/duckduckgo-autofill@12.2.0...13.1.0

## Steps to test
This release has been tested during autofill development. For smoke test
steps see [this
task](https://app.asana.com/0/1198964220583541/1200583647142330/f).

Co-authored-by: dbajpeyi <3018923+dbajpeyi@users.noreply.github.com>
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.

2 participants