[Credentials Import] Check local credentials before showing prompt (#650

) * chore: remove response * chore: rename pixel * feat: update copy * feat: update locales * fix: only show tooltip if there's no credentials locally stored
duckduckgo · Sep 9, 2024 · d879b6c · d879b6c
1 parent bd62c42
commit d879b6c
Show file tree

Hide file tree

Showing 5 changed files with 10 additions and 5 deletions.
diff --git a/dist/autofill-debug.js b/dist/autofill-debug.js
diff --git a/dist/autofill.js b/dist/autofill.js
diff --git a/src/UI/controllers/HTMLTooltipUIController.js b/src/UI/controllers/HTMLTooltipUIController.js
@@ -109,7 +109,8 @@ export class HTMLTooltipUIController extends UIController {
 
         }
 
-        if (topContextData.credentialsImport) {
+        const hasNoCredentialsData = this._options.device.getLocalCredentials().length === 0
+        if (topContextData.credentialsImport && hasNoCredentialsData) {
             this._options.device.firePixel({pixelName: 'autofill_import_credentials_prompt_shown'})
             return new CredentialsImportTooltip(config, topContextData.inputType, getPosition, tooltipOptions)
                 .render(this._options.device, {

diff --git a/swift-package/Resources/assets/autofill-debug.js b/swift-package/Resources/assets/autofill-debug.js
diff --git a/swift-package/Resources/assets/autofill.js b/swift-package/Resources/assets/autofill.js