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

[Credentials Import] remove API response #648

Merged
merged 5 commits into from
Sep 6, 2024

Conversation

dbajpeyi
Copy link
Collaborator

@dbajpeyi dbajpeyi commented Sep 6, 2024

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

Description

Quick follow-up to #631

  1. We won't need the response in startCredentialsImportFlowCall, since now it's only a notify call.
  2. Also aligns pixel name to https://app.asana.com/0/69071770703008/1208167249157400/f

Steps to test

Currently BSK is broken/in-progress so couldn't test it, but Gabor confirmed on Windows that all is good. We were not using the response anyway.

@dbajpeyi dbajpeyi changed the title chore: remove response [Credentials Import] remove API response Sep 6, 2024
@dbajpeyi dbajpeyi force-pushed the dbajpeyi/remove-response-param branch from 5faacf1 to 38bcdc8 Compare September 6, 2024 09:29
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.

LGTM! 🚀

@dbajpeyi
Copy link
Collaborator Author

dbajpeyi commented Sep 6, 2024

Screenshot 2024-09-06 at 17 33 37 Screenshot 2024-09-06 at 17 33 02 Screenshot 2024-09-06 at 17 32 35

Quick test with some of the locales, looks good! Merging it

@dbajpeyi dbajpeyi merged commit bd62c42 into main Sep 6, 2024
1 check passed
@dbajpeyi dbajpeyi deleted the dbajpeyi/remove-response-param branch September 6, 2024 15:34
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