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

Add clear rewards data and clear payment data #926

Closed
NejcZdovc opened this issue Sep 2, 2018 · 20 comments · Fixed by brave/brave-core#6020
Closed

Add clear rewards data and clear payment data #926

NejcZdovc opened this issue Sep 2, 2018 · 20 comments · Fixed by brave/brave-core#6020

Comments

@NejcZdovc
Copy link
Contributor

NejcZdovc commented Sep 2, 2018

User should be able to remove rewards data (auto contribute, donation, tipping) and remove payment data (previous payments). This should delete the data on disk.

From browser laptop
image

@NejcZdovc NejcZdovc added this to the Releasable builds 0.55.x milestone Sep 2, 2018
@bbondy bbondy modified the milestones: Releasable builds 0.55.x, 1.0 Sep 3, 2018
@rebron rebron added the feature/global-settings Settings at browser level independent of shields settings label Oct 16, 2018
@NejcZdovc NejcZdovc modified the milestones: 1.0, 1.x Backlog Oct 22, 2018
@NejcZdovc NejcZdovc added the priority/P5 Not scheduled. Don't anticipate work on this any time soon. label Oct 31, 2018
@diracdeltas diracdeltas added priority/P3 The next thing for us to work on. It'll ride the trains. and removed priority/P5 Not scheduled. Don't anticipate work on this any time soon. labels Nov 27, 2018
@diracdeltas
Copy link
Member

Marking this as p3 and privacy related because currently there is no way to delete browsing history associated with rewards unless the user deletes the files on disk. if rewards is enabled, browsing history is logged to publisher_info_db. @NejcZdovc please correct me if i'm wrong.

cc @tomlowenthal

@tildelowengrimm
Copy link
Contributor

I think the right solution would be to extend brave://settings/clearBrowserData to include controls for Brave Rewards.

For the simply view, we should add a "Brave Rewards data" item which removes all attention info created from the indicated time period.

On the advanced tab, we should add a couple of items.

  • "Brave Rewards auto-contribute list" should remove attention history and auto-contribute items for the applicable time period.
  • "Brave Rewards contribution history" should remove records of tips and contributions from the time period, replacing them with "private contribution" or something to that effect.

@mandar-brave @NejcZdovc Could someone on the Rewards team indicate whether there are any other ways that information equivalent to browsing-history might be stored?

@tildelowengrimm tildelowengrimm added priority/P2 A bad problem. We might uplift this to the next planned release. and removed priority/P3 The next thing for us to work on. It'll ride the trains. labels Nov 28, 2018
@jasonrsadler
Copy link

What is defined as "applicable time period"? Currently we store auto-contribute information based on month and year. The options available when clearing data are "last hour", "last 24 hours", "last 7 days", and "last 4 weeks". How should we handle clearing data based on what is chosen?

@tildelowengrimm
Copy link
Contributor

@jasonrsadler Are you saying that we have no way of knowing what auto-contribute info was added in the last hour, the last 24 hours, or last 7 days?

@jasonrsadler
Copy link

Correct. @NejcZdovc to confirm but I believe we are only storing the month and the year.

@NejcZdovc
Copy link
Contributor Author

@tomlowenthal that is correct we accumulate time per month and not per visit

@tildelowengrimm
Copy link
Contributor

Sounds like we'll need to change that in order to be able to support deletion.

@diracdeltas
Copy link
Member

Personally I would be happy if someone just added an option for "Clear Brave rewards and payment data since the beginning of time".

Having time intervals can be a follow-up but I don't think it's strictly necessary (we didn't have this in b-l)

@NejcZdovc
Copy link
Contributor Author

I think we shouldn't change it. We should adjust place where we have clear like @diracdeltas said

@LaurenWags
Copy link
Member

LaurenWags commented Aug 14, 2020

Verified passed with

Brave | 1.13.71 Chromium: 84.0.4147.125 (Official Build) dev (64-bit)
-- | --
Revision | d0784639447f2e10d32ebaf9861092b20cfde286-refs/branch-heads/4147@{#1059}
OS | macOS Version 10.14.6 (Build 18G3020)

Screen Shot 2020-08-14 at 10 27 57 AM

Screen Shot 2020-08-14 at 10 28 10 AM

Screen Shot 2020-08-14 at 10 28 20 AM

  • Confirmed clicking on the link from each tab opens the Manage Your Wallet dialog

Screen Shot 2020-08-14 at 10 28 50 AM

  • Confirmed link displays using light theme also:

Screen Shot 2020-08-14 at 10 30 13 AM


Verification passed on

Brave 1.13.71 Chromium: 84.0.4147.125 (Official Build) dev (64-bit)
Revision d0784639447f2e10d32ebaf9861092b20cfde286-refs/branch-heads/4147@{#1059}
OS Ubuntu 18.04 LTS
  • Verified test plan from Adds UI to clear browsing data flow brave-core#6020

  • Confirmed link exists on each Clear browsing data tab(brave://settings/clearBrowserData) :
    Dark theme
    image
    image
    image
    Light Theme
    image

  • Confirmed clicking on the link from each tab opens the Manage Your Wallet dialog
    image

Encountered #11235
Logged #11244


Verification passed on

Brave | 1.13.73 Chromium: 84.0.4147.125 (Official Build) dev (64-bit)
-- | --
Revision | d0784639447f2e10d32ebaf9861092b20cfde286-refs/branch-heads/4147@{#1059}
OS | Windows 10 OS Version 1903 (Build 18362.1016)

Basic View:
image

Advanced view:
image

On exit:
image

  • Confirmed clicking on the link from each tab opens the Manage Your Wallet dialog
    image
  • Confirmed link displays using Dark theme also:
    image

-Encountered #11235
-Verified Reset wallet works as expected

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment