For #18711: Telemetry for credit card autofill #19548
For #18711: Telemetry for credit card autofill #19548
Conversation
7d393b0
to
814cc79
Compare
814cc79
to
82df883
Compare
Request for data collection review formAll questions are mandatory. You must receive review from a data steward peer on your responses to these questions before shipping new data collection.
|
82df883
to
4b7b913
Compare
4b7b913
to
2b7b80b
Compare
Codecov Report
@@ Coverage Diff @@
## master #19548 +/- ##
============================================
- Coverage 35.42% 35.42% -0.01%
- Complexity 1648 1649 +1
============================================
Files 545 545
Lines 22030 22045 +15
Branches 3267 3271 +4
============================================
+ Hits 7805 7809 +4
- Misses 13326 13334 +8
- Partials 899 902 +3
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me. Some small nits. 👍
- interaction | ||
notification_emails: | ||
- fenix-core@mozilla.com | ||
expires: "2021-10-01" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: the expiry date might be too close?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It can be easily extended later if need-be. The data-review form for extending a collection is even shorter to help facilitate this.
...rc/test/java/org/mozilla/fenix/settings/creditcards/DefaultCreditCardEditorControllerTest.kt
Show resolved
Hide resolved
Data Review
Data Review ResultApproved: data-review+ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Data-Review approval only, this doesn't cover any code review from me.
app/src/main/java/org/mozilla/fenix/autofill/AutofillConfirmActivity.kt
Outdated
Show resolved
Hide resolved
...rc/main/java/org/mozilla/fenix/settings/creditcards/controller/CreditCardEditorController.kt
Outdated
Show resolved
Hide resolved
...rc/main/java/org/mozilla/fenix/settings/creditcards/controller/CreditCardEditorController.kt
Outdated
Show resolved
Hide resolved
...rc/main/java/org/mozilla/fenix/settings/creditcards/controller/CreditCardEditorController.kt
Outdated
Show resolved
Hide resolved
This is waiting on a |
...rc/main/java/org/mozilla/fenix/settings/creditcards/controller/CreditCardEditorController.kt
Outdated
Show resolved
Hide resolved
@eliserichards If there's nothing blocking this except for the AC bump, let's try to land some of probes and get a head start on the QA. We will probably want to do another review for the new facts. |
This pull request has conflicts when rebasing. Could you fix it @eliserichards? 🙏 |
Separate autofill save from manual save for credit card telemetry Descoped autofill save
Add test context for controller test
fc1a6b6
to
2c79736
Compare
…ozilla-mobile#19548)" This reverts commit e0b410e.
This reverts commit e0b410e.
…ozilla-mobile#19548)" This reverts commit e0b410e.
…ozilla-mobile#19548)" This reverts commit e0b410e.
For #18711
Waiting on DS confirmation: https://jira.mozilla.com/browse/DS-1598✅Needs data review:#19548 (comment) ✅AC Facts for autofill:mozilla-mobile/android-components#10342 ✅Pull Request checklist
To download an APK when reviewing a PR: