Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

Multiple issues with Autofill #4155

Closed
srirambv opened this issue Sep 21, 2016 · 0 comments · Fixed by #4179
Closed

Multiple issues with Autofill #4155

srirambv opened this issue Sep 21, 2016 · 0 comments · Fixed by #4179
Assignees
@darkdh
Labels
feature/autofill QA/checked-Linux QA/checked-macOS QA/checked-Win32 QA/checked-Win64
Milestone
0.12.2dev

Comments

@srirambv
Copy link
Collaborator

srirambv commented Sep 21, 2016 •
edited
Loading

Did you search for similar issues before submitting this one?
Yes

Describe the issue you encountered:
Autofill doesn't store Country Name,
Duplicates new address
Deletes all contents when one is deleted

Expected behavior:
Should store Country details,
Should not duplicate records
Should only delete the selected one

  • Platform (Win7, 8, 10? macOS? Linux distro?):
    Windows 10 x64

  • Brave Version:
    0.12.2 Dev RC1

  • Steps to reproduce:

    1. Open about:autofill
    2. Add new Address, Country is not saved
    3. Edit entry add Country again and save, doesn't store
    4. Add new entry with just one field, duplicates the previous entry
    5. Delete one entry deletes all entries

    Same issue with Credit Card info as well

  • Screenshot if needed:
    0

  • Any related issues:

cc: @darkdh
@srirambv srirambv added this to the 0.12.2dev milestone Sep 21, 2016
@darkdh darkdh self-assigned this Sep 22, 2016
darkdh added a commit to darkdh/browser-laptop that referenced this issue Sep 22, 2016
@darkdh
Autofill data add/edit method refactoring
dd3efea 
1. add/edit data on default session
2. Apply changes to brave/muon#61

requires brave/muon#61

fix brave#4155
fix brave#3725

Auditors: @bridiver, @bbondy

Test Plan:
STR in brave#4155 and brave#3725
@darkdh darkdh mentioned this issue Sep 22, 2016
Merged
4 tasks
@luixxiul luixxiul mentioned this issue Sep 22, 2016
Closed
This was referenced Sep 22, 2016
Closed
Closed
Closed
Closed
This was referenced Sep 24, 2016
Closed
Closed
Closed
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@darkdh darkdh

Labels
feature/autofill QA/checked-Linux QA/checked-macOS QA/checked-Win32 QA/checked-Win64
Projects
None yet
Milestone
0.12.2dev
Development

Successfully merging a pull request may close this issue.

Autofill data add/edit method refactoring darkdh/browser-laptop
4 participants
@aekeus @alexwykoff @darkdh @srirambv