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

Commit

Permalink
Fix advanced settings panel; when renaming was done, one instance was…
Browse files Browse the repository at this point in the history
… missed with be048b8

Fixes #11260

Auditors: @luixxiul

Test Plan:
1. Visit about:preferences and open the payments tab
2. Enable payments and click the gear in the top right (settings)
3. Change visit to something other than its existing value
4. Exit and re-open advanced settings again
5. Setting retained its value
  • Loading branch information
bsclifton committed Oct 4, 2017
1 parent 08781a5 commit d266f64
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,7 @@ class AdvancedSettingsContent extends ImmutableComponent {
<SettingDropdown
data-test-id='visitSelector'
defaultValue={minPublisherVisits || 1}
onChange={changeSetting.bind(null, this.props.onChangeSetting, settings.MINIMUM_VISITS)}>
onChange={changeSetting.bind(null, this.props.onChangeSetting, settings.PAYMENTS_MINIMUM_VISITS)}>
<option data-l10n-id='minimumVisitsLow' value='1' />
<option data-l10n-id='minimumVisitsMedium' value='5' />
<option data-l10n-id='minimumVisitsHigh' value='10' />
Expand Down

1 comment on commit d266f64

@luixxiul
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

++

Please sign in to comment.