Fixes brave-browser/issues#2273

Fixes brave-browser/issues#2332 Triggers balance report fetch onWalletProperties, this is so that reports update along with wallet info Also ensures that the current report is fetched on settings mount
brave · Dec 18, 2018 · 12342c9 · 12342c9
1 parent b345d6b
commit 12342c9
Show file tree

Hide file tree

Showing 5 changed files with 12 additions and 1 deletion.
diff --git a/components/brave_rewards/browser/rewards_service_impl.cc b/components/brave_rewards/browser/rewards_service_impl.cc
@@ -606,6 +606,7 @@ void RewardsServiceImpl::OnGrantFinish(ledger::Result result,
  grant.probi);
  }
 
+ GetCurrentBalanceReport();
  TriggerOnGrantFinish(result, grant);
 }
 
@@ -624,6 +625,7 @@ void RewardsServiceImpl::OnReconcileComplete(ledger::Result result,
  GetCurrentTimestamp());
  }
 
+ GetCurrentBalanceReport();
  for (auto& observer : observers_)
  observer.OnReconcileComplete(this,
  result,

diff --git a/components/brave_rewards/resources/ui/actions/rewards_actions.ts b/components/brave_rewards/resources/ui/actions/rewards_actions.ts
@@ -93,6 +93,8 @@ export const onBalanceReports = (reports: Record<string, Rewards.Report>) => act
  reports
 })
 
+export const getCurrentReport = () => action(types.GET_CURRENT_REPORT, {})
+
 export const excludePublisher = (publisherKey: string) => action(types.ON_EXCLUDE_PUBLISHER, {
  publisherKey
 })

diff --git a/components/brave_rewards/resources/ui/components/settingsPage.tsx b/components/brave_rewards/resources/ui/components/settingsPage.tsx
@@ -52,6 +52,7 @@ class SettingsPage extends React.Component<Props, {}> {
  this.actions.getContributeList()
  this.actions.checkImported()
  this.actions.getAdsData()
+ this.actions.getCurrentReport()
  }
 
  componentWillUnmount () {

diff --git a/components/brave_rewards/resources/ui/constants/rewards_types.ts b/components/brave_rewards/resources/ui/constants/rewards_types.ts
@@ -46,5 +46,6 @@ export const enum types {
  ON_IMPORTED_CHECK = '@@rewards/ON_IMPORTED_CHECK',
  GET_ADS_DATA = '@@rewards/GET_ADS_DATA',
  ON_ADS_DATA = '@@rewards/ON_ADS_DATA',
- ON_ADS_SETTING_SAVE = '@@rewards/ON_ADS_SETTING_SAVE'
+ ON_ADS_SETTING_SAVE = '@@rewards/ON_ADS_SETTING_SAVE',
+ GET_CURRENT_REPORT = '@@rewards/GET_CURRENT_REPORT'
 }
diff --git a/components/brave_rewards/resources/ui/reducers/wallet_reducer.ts b/components/brave_rewards/resources/ui/reducers/wallet_reducer.ts
@@ -170,6 +170,11 @@ const walletReducer: Reducer<Rewards.State | undefined> = (state: Rewards.State,
  }
  break
  }
+ case types.GET_CURRENT_REPORT:
+ {
+ chrome.send('brave_rewards.getBalanceReports')
+ break
+ }
  case types.ON_BALANCE_REPORTS:
  {
  state = { ...state }