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

Commit

Permalink
Merge pull request #6972 from brave/issue-6941
Browse files Browse the repository at this point in the history
Minimum publisher visit default now 1
  • Loading branch information
bsclifton authored Feb 1, 2017
2 parents 4ea3be9 + ee556bd commit 296c6d6
Showing 3 changed files with 4 additions and 4 deletions.
2 changes: 1 addition & 1 deletion app/ledger.js
Original file line number Diff line number Diff line change
@@ -761,7 +761,7 @@ var enable = (paymentsEnabled) => {

value = getSetting(settings.MINIMUM_VISITS)
if (!value) {
value = 5
value = 1
appActions.changeSetting(settings.MINIMUM_VISITS, value)
}
if (value > 0) synopsis.options.minPublisherVisits = value
4 changes: 2 additions & 2 deletions app/renderer/components/preferences/paymentsTab.js
Original file line number Diff line number Diff line change
@@ -251,9 +251,9 @@ class PaymentsTab extends ImmutableComponent {
<SettingItem>
<SettingDropdown
data-test-id='visitSelector'
defaultValue={minPublisherVisits || 5}
defaultValue={minPublisherVisits || 1}
onChange={changeSetting.bind(null, this.props.onChangeSetting, settings.MINIMUM_VISITS)}>>>
<option value='2'>2 visits</option>
<option value='1'>1 visits</option>
<option value='5'>5 visits</option>
<option value='10'>10 visits</option>
</SettingDropdown>
2 changes: 1 addition & 1 deletion js/constants/appConfig.js
Original file line number Diff line number Diff line change
@@ -155,7 +155,7 @@ module.exports = {
'advanced.send-usage-statistics': false,
'advanced.hide-excluded-sites': false,
'advanced.minimum-visit-time': 8,
'advanced.minimum-visits': 5,
'advanced.minimum-visits': 1,
'advanced.minimum-percentage': false,
'advanced.auto-suggest-sites': true,
'shutdown.clear-history': false,

0 comments on commit 296c6d6

Please sign in to comment.