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

Add "Clear Payment Data / Clear Payment history" option to about:preferences#security and delete wallet. #8537

Closed
mrose17 opened this issue Apr 27, 2017 · 6 comments

Comments

@mrose17
Copy link
Member

mrose17 commented Apr 27, 2017

Test plan

defined in #14116 (comment)

  • Did you search for similar issues before submitting this one? yes

  • Describe the issue you encountered: about:preferences#sync has an option to let the user completely erase the sync data; we need a similar option for the ledger

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

  • Brave Version (revision SHA): n/a

  • Steps to reproduce:

    1. go to about:preferences#payments
    2. look for a "Reset Ledger" option
    3. look in about:preferences#security and in History > 'Clear Browsing Data...' for a 'Clear payments' option
  • Actual result: you won't find it

  • Expected result:

    • clicking on the button in 'about:preferences#payments' will result in one of two popups:
    1. if the balance of the wallet is zero, then it should look like the popup for "Reset Sync"
    2. otherwise, the user should be given the option of "Yes", "Keep wallet", or "Cancel"
    • clicking on the switch in the Clear Browsing Data panel will clear the ledger synopsis (and payment history?)
  • Will the steps above reproduce in a fresh profile? no, If not what other info can be added? you have to enable the ledger first

  • Is this an issue in the currently released version? yes

  • Can this issue be consistently reproduced? yes

  • Extra QA steps: none

  • Screenshot if needed: n/a

  • Any related issues: tangentially related to Auto-purging "Saved site settings" resets Payments "Include" toggles #6708

Note to @bsclifton and @NejcZdovc:

  1. to do a complete reset, set settings.PAYMENTS_ENABLED to false and then call reset
  2. a partial reset ("Keep wallet") is actually pretty easy to do, let me know when we're ready to implement that, thanks!
@bsclifton bsclifton added this to the 0.15.3 milestone May 2, 2017
@alexwykoff alexwykoff modified the milestones: 0.16.200, 0.16.100 May 23, 2017
@bbondy bbondy modified the milestones: 0.19.x (Nightly Channel), 0.18.x (Developer Channel) Jun 13, 2017
@alexwykoff alexwykoff modified the milestones: 0.20.x (Developer Channel), 0.19.x (Beta Channel) Jul 18, 2017
@NejcZdovc NejcZdovc modified the milestones: 0.21.x (Nightly Channel), 0.20.x (Developer Channel) Jul 20, 2017
@diracdeltas
Copy link
Member

I have personally gotten 2 reports from people who think that payments should be cleared when 'Clear History' is clicked. I'd also expect this behavior. Is there a reason not to do that?

@diracdeltas
Copy link
Member

Alternatively, it might be good enough to just have a 'Clear payments data' switch in about:preferences#security and in History > 'Clear Browsing Data...'

@mrose17
Copy link
Member Author

mrose17 commented Aug 16, 2017

that works for me.

@diracdeltas diracdeltas changed the title Add "Clear Data / Reset Ledger" option to about:preferences#payments Add "Clear Data / Reset Ledger" option to about:preferences#payments and 'Clear browsing data...' panel Aug 16, 2017
@diracdeltas
Copy link
Member

Specifically i think the switch in the Clear Browsing Data panel should only be for clearing synopsis, not resetting the wallet. The button in 'about:preferences#payments' should offer the option to reset the wallet, however.

@bradleyrichter
Copy link
Contributor

Here is the proposed switch position and string:

image

The switch follows the normal switch implementation but adds the (?) button at the end which will lead to a git hub wiki page that explains how the data is recorded separately from history as needed by the ledger system.

cc: @mrose17 cc: @diracdeltas

@bbondy bbondy modified the milestones: 0.21.x (Developer Channel), 0.20.x (Beta Channel), Backlog Oct 25, 2017
@alexwykoff alexwykoff added the priority/P4 Minor loss of function. Workaround usually present. label Oct 31, 2017
@bbondy bbondy modified the milestones: Triage Backlog, Prioritized Backlog Nov 2, 2017
@NejcZdovc NejcZdovc added the post-v1 We don't expect to be able to resolve this before releasing v1.0 with Brave Core (instead of Muon). label Apr 6, 2018
@NejcZdovc NejcZdovc removed the post-v1 We don't expect to be able to resolve this before releasing v1.0 with Brave Core (instead of Muon). label May 14, 2018
NejcZdovc added a commit to NejcZdovc/browser-laptop that referenced this issue May 14, 2018
Resolves brave#8537

Auditors:

Test Plan:
@NejcZdovc NejcZdovc modified the milestones: Backlog (Prioritized), 0.22.x Release 5 May 22, 2018
NejcZdovc added a commit to NejcZdovc/browser-laptop that referenced this issue May 24, 2018
Resolves brave#8537

Auditors:

Test Plan:
NejcZdovc added a commit to NejcZdovc/browser-laptop that referenced this issue May 25, 2018
Resolves brave#8537

Auditors:

Test Plan:
NejcZdovc added a commit to NejcZdovc/browser-laptop that referenced this issue May 28, 2018
Resolves brave#8537

Auditors:

Test Plan:
NejcZdovc added a commit to NejcZdovc/browser-laptop that referenced this issue May 28, 2018
Resolves brave#8537

Auditors:

Test Plan:
NejcZdovc added a commit to NejcZdovc/browser-laptop that referenced this issue May 28, 2018
Resolves brave#8537

Auditors:

Test Plan:
NejcZdovc added a commit to NejcZdovc/browser-laptop that referenced this issue May 28, 2018
Resolves brave#8537

Auditors:

Test Plan:
NejcZdovc added a commit to NejcZdovc/browser-laptop that referenced this issue May 28, 2018
Resolves brave#8537

Auditors:

Test Plan:
NejcZdovc added a commit to NejcZdovc/browser-laptop that referenced this issue May 29, 2018
Resolves brave#8537

Auditors:

Test Plan:
NejcZdovc added a commit to NejcZdovc/browser-laptop that referenced this issue May 29, 2018
Resolves brave#8537

Auditors:

Test Plan:
NejcZdovc added a commit to NejcZdovc/browser-laptop that referenced this issue May 29, 2018
Resolves brave#8537

Auditors:

Test Plan:
NejcZdovc added a commit to NejcZdovc/browser-laptop that referenced this issue May 29, 2018
Resolves brave#8537

Auditors:

Test Plan:
NejcZdovc added a commit that referenced this issue May 29, 2018
NejcZdovc added a commit that referenced this issue May 29, 2018
NejcZdovc added a commit to NejcZdovc/browser-laptop that referenced this issue May 29, 2018
@srirambv
Copy link
Collaborator

srirambv commented May 30, 2018

Verified on Windows x64 using

  • 0.22.723 edb3e88
  • muon 6.0.12
  • libchromiumcontent 66.0.3359.170
  • Verified publisher removed via the clear history is getting re-added
  • Verified the settings is disabled when contribution is in progress

Verified with macOS 10.12.6 using

  • 0.22.723 edb3e88
  • muon 6.0.12
  • libchromiumcontent 66.0.3359.170

Verified on Ubuntu 17.10 x64

  • 0.22.723 edb3e88
  • muon 6.0.12
  • libchromiumcontent 66.0.3359.170

@kjozwiak kjozwiak changed the title Add "Clear Data / Reset Ledger" option to about:preferences#payments and 'Clear browsing data...' panel Add "Clear Payment Data / Clear Payment history" option to about:preferences#security and delete wallet. Jun 4, 2018
ryanml added a commit to ryanml/browser-laptop that referenced this issue Jun 22, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.