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

Add link on about:preferences#payments panel to Brave Payments Contributors Terms of Service #11942

Closed
mrose17 opened this issue Nov 14, 2017 · 8 comments · Fixed by #11957
Closed

Comments

@mrose17
Copy link
Member

mrose17 commented Nov 14, 2017

1

as noted in the mockup above, there should be a link on about:preferences#payments labelled "Terms of Service" that opens a new tab at https://basicattentiontoken.org/contributor-terms-of-service/

New mock for ledger table

1

@bsclifton
Copy link
Member

Moving back to Hotfix 6; if we can get a PR for this today, reviewer can decide on whether or not to pull into Hotfix 5 (since we need a release candidate today)

@cezaraugusto
Copy link
Contributor

Opened #11957 moving back to hotfix 5

@srirambv
Copy link
Collaborator

Not as per mockup spec. Terms of service link is shown below the table.
image

@srirambv srirambv reopened this Nov 15, 2017
@NejcZdovc
Copy link
Contributor

@srirambv mock was updated and you can find it here #11957 (comment). Will update this issue as well

@kjozwiak
Copy link
Member

Currently, the ToS only appears in the wallet when there's entries within the the ledger table... However, the ToS are not being displayed in the following cases:

  • initial wallet creation
  • empty ledger table

After a quick discussion with @mrose17, the ToS should be visible at all times.

STR:

  • install brave and create a wallet (no ToS being displayed)
  • visit a website and popular the ledger table (ToS visible)
  • remove the entry from the table (no ToS being displayed)

tosexample

@luixxiul
Copy link
Contributor

If TOS should be displayed even before enabling the payments, the link should be displayed just above the closing div tag.

{
this.enabled
? <EnabledContent settings={this.props.settings}
onChangeSetting={this.props.onChangeSetting}
ledgerData={this.props.ledgerData}
showOverlay={this.props.showOverlay}
siteSettings={this.props.siteSettings}
/>
: <DisabledContent />
}
</div>

@NejcZdovc
Copy link
Contributor

@kjozwiak I think that need to display it only when wallet is enabled. Can you please confirm this?

@mrose17
Copy link
Member Author

mrose17 commented Nov 15, 2017

@NejcZdovc - it's needs to be there even when the wallet is enabled. it is entirely reasonable for someone to want to look at the terms of service before opting in. we need to give them that opportunity.

of course, "nobody reads those things" (or so i'm told), but it still needs to be available to allow people to make a fully informed decision, if they so chose.

that's one reason why the publishers.basicattentiontoken.org has a link to the publishers' terms of service on the home page, even if you haven't logged in...

cezaraugusto added a commit that referenced this issue Nov 15, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.