Skip to content
This repository has been archived by the owner on Feb 20, 2023. It is now read-only.

[Saved cards] Display the obfuscated credit card numbers in Saved cards #18270

Closed
gabrielluong opened this issue Mar 2, 2021 · 4 comments · Fixed by #19420
Closed

[Saved cards] Display the obfuscated credit card numbers in Saved cards #18270

gabrielluong opened this issue Mar 2, 2021 · 4 comments · Fixed by #19420
Assignees
Labels
cc-autofill Autofill credit card E1 Estimation Point: trivial, less than half day eng:qa:verified QA Verified Feature:Autofill Address and Credit Card autofill
Milestone

Comments

@gabrielluong
Copy link
Member

gabrielluong commented Mar 2, 2021

See #18270 (comment) for the latest.

When mozilla-mobile/android-components#9804 is available, we should use obfuscatedCardNumber provided when display the obfuscated credit card numbers in the "Saved cards" screen.

This depends on #18268 and mozilla-mobile/android-components#9804

Screen Shot 2021-03-02 at 3 39 09 PM

┆Issue is synchronized with this Jira Task

@gabrielluong gabrielluong added E1 Estimation Point: trivial, less than half day Feature:Autofill Address and Credit Card autofill cc-autofill Autofill credit card labels Mar 2, 2021
@github-actions github-actions bot added the needs:triage Issue needs triage label Mar 2, 2021
@gabrielluong gabrielluong added blocked and removed needs:triage Issue needs triage labels Mar 2, 2021
@gabrielluong gabrielluong changed the title Display the obfuscated credit card numbers in Saved cards [Saved cards] Display the obfuscated credit card numbers in Saved cards Mar 2, 2021
@gabrielluong gabrielluong added the needs:ac Needs Android Component Work label Mar 2, 2021
@gabrielluong gabrielluong removed blocked needs:ac Needs Android Component Work labels Apr 26, 2021
@gabrielluong
Copy link
Member Author

gabrielluong commented Apr 26, 2021

Currently, we show the encrypted card number here

We should change this to display the cardNumberLast4 https://github.com/mozilla-mobile/android-components/blob/7425e13d075f737f548e3cecd04446628afa8c12/components/concept/storage/src/main/java/mozilla/components/concept/storage/CreditCardsAddressesStorage.kt#L147

In doing so, we should also add the necessary 4 dots ellipsis in front of the last 4 card number. See Chromium's unicode that is used for 4 dots ellipsis. https://github.com/chromium/chromium/blob/d7da0240cae77824d1eda25745c4022757499131/chrome/android/javatests/src/org/chromium/chrome/browser/autofill/PersonalDataManagerTest.java#L153-L159. We can potentially just add the unicode to the CreditCardEditorState here as the cardNumber. Alternatively, we should also highly consider adding a helper method in AC to provide this obfuscated card number (last 4 card number + 4 dots ellipsis) so that it can be used everywhere including the select credit card prompts.

We should examine the 4 dots ellipsis used in Chromium and see what we do in Edit Logins to see if we should align. Otherwise, I am interested to know if the \u202A is the Left-To-Right Embedding (LTE) mark, and // \u202C is the Pop Directional Formatting (PDF) mark are helpful for RTL and different locale support.

We can't show "Visa", "Amex", the credit card issuers yet until mozilla-mobile/android-components#9813 is done. So, that will be out of scope. We just want to display the obfuscated card number in this issue and clean up our CreditCardEditorState state with this issue. Please also update the relevant tests.

@codrut-topliceanu codrut-topliceanu self-assigned this Apr 29, 2021
codrut-topliceanu pushed a commit to codrut-topliceanu/fenix that referenced this issue May 7, 2021
@gabrielluong gabrielluong added this to the 90 milestone May 7, 2021
@eliserichards eliserichards linked a pull request May 10, 2021 that will close this issue
3 tasks
codrut-topliceanu pushed a commit to codrut-topliceanu/fenix that referenced this issue May 12, 2021
codrut-topliceanu pushed a commit to codrut-topliceanu/fenix that referenced this issue May 13, 2021
codrut-topliceanu pushed a commit to codrut-topliceanu/fenix that referenced this issue May 13, 2021
codrut-topliceanu pushed a commit to codrut-topliceanu/fenix that referenced this issue May 13, 2021
gabrielluong pushed a commit to gabrielluong/fenix that referenced this issue May 24, 2021
gabrielluong pushed a commit to gabrielluong/fenix that referenced this issue May 24, 2021
gabrielluong pushed a commit to gabrielluong/fenix that referenced this issue May 24, 2021
gabrielluong pushed a commit to gabrielluong/fenix that referenced this issue May 24, 2021
gabrielluong pushed a commit to gabrielluong/fenix that referenced this issue May 24, 2021
codrut-topliceanu pushed a commit to codrut-topliceanu/fenix that referenced this issue May 25, 2021
codrut-topliceanu pushed a commit to codrut-topliceanu/fenix that referenced this issue May 25, 2021
codrut-topliceanu pushed a commit to codrut-topliceanu/fenix that referenced this issue May 25, 2021
codrut-topliceanu pushed a commit to codrut-topliceanu/fenix that referenced this issue May 25, 2021
codrut-topliceanu pushed a commit to codrut-topliceanu/fenix that referenced this issue May 25, 2021
codrut-topliceanu pushed a commit to codrut-topliceanu/fenix that referenced this issue May 25, 2021
codrut-topliceanu added a commit that referenced this issue May 25, 2021
* For #18270 - Adds ellipses+last 4 digits to CC display
@gabrielluong gabrielluong added the eng:qa:needed QA Needed label May 25, 2021
@abodea
Copy link
Member

abodea commented May 26, 2021

Verified as fixed on the latest Nightly 5/26 with OnePlus Nord (Android 10).
Note that the icons are still not displayed but I mentioned this issue here #18279 (comment).

And the credit card number is still displayed #19482 Encrypted... but that bug was close in favor of this one.

So I was thinking should we re-open because when tapping on a credit card the encrypted number is still displayed, or should I open a new bug?

@abodea abodea closed this as completed May 26, 2021
@abodea abodea added eng:qa:verified QA Verified and removed eng:qa:needed QA Needed labels May 26, 2021
@gabrielluong
Copy link
Member Author

Verified as fixed on the latest Nightly 5/26 with OnePlus Nord (Android 10).
Note that the icons are still not displayed but I mentioned this issue here #18279 (comment).

And the credit card number is still displayed #19482 Encrypted... but that bug was close in favor of this one.

So I was thinking should we re-open because when tapping on a credit card the encrypted number is still displayed, or should I open a new bug?

We got mozilla-mobile/android-components#10333 for displaying the decrypted card number

@abodea
Copy link
Member

abodea commented May 26, 2021

Verified as fixed on the latest Nightly 5/26 with OnePlus Nord (Android 10).
Note that the icons are still not displayed but I mentioned this issue here #18279 (comment).
And the credit card number is still displayed #19482 Encrypted... but that bug was close in favor of this one.
So I was thinking should we re-open because when tapping on a credit card the encrypted number is still displayed, or should I open a new bug?

We got mozilla-mobile/android-components#10333 for displaying the decrypted card number

Thank you @gabrielluong thank I think we are all good on this side, waiting on that to be fixed so I will confirm it!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cc-autofill Autofill credit card E1 Estimation Point: trivial, less than half day eng:qa:verified QA Verified Feature:Autofill Address and Credit Card autofill
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants