Removes date from ads grants in the wallet

Resolves brave/brave-browser#4169
brave · Apr 26, 2019 · c3f2ea0 · c3f2ea0
1 parent af76b8e
commit c3f2ea0
Show file tree

Hide file tree

Showing 16 changed files with 87 additions and 84 deletions.
diff --git a/browser/ui/webui/brave_donate_ui.cc b/browser/ui/webui/brave_donate_ui.cc
@@ -174,6 +174,7 @@ void RewardsDonateDOMHandler::OnWalletProperties(
       auto grant = std::make_unique<base::DictionaryValue>();
       grant->SetString("probi", item.probi);
       grant->SetInteger("expiryTime", item.expiryTime);
+      grant->SetString("type", item.type);
       grants->Append(std::move(grant));
     }
     walletInfo->SetList("grants", std::move(grants));

diff --git a/browser/ui/webui/brave_rewards_ui.cc b/browser/ui/webui/brave_rewards_ui.cc
@@ -425,6 +425,7 @@ void RewardsDOMHandler::OnGetAutoContributeProps(
         auto grant = std::make_unique<base::DictionaryValue>();
         grant->SetString("probi", item.probi);
         grant->SetInteger("expiryTime", item.expiryTime);
+        grant->SetString("type", item.type);
         grants->Append(std::move(grant));
       }
       walletInfo->SetList("grants", std::move(grants));
@@ -545,6 +546,7 @@ void RewardsDOMHandler::OnRecoverWallet(
       auto grant = std::make_unique<base::DictionaryValue>();
       grant->SetString("probi", item.probi);
       grant->SetInteger("expiryTime", item.expiryTime);
+      grant->SetString("type", item.type);
       newGrants->Append(std::move(grant));
     }
     recover.SetList("grants", std::move(newGrants));

diff --git a/browser/ui/webui/brave_webui_source.cc b/browser/ui/webui/brave_webui_source.cc
@@ -270,6 +270,7 @@ void CustomizeWebUIHTMLSource(const std::string &name,
         { "addFundsQR", IDS_BRAVE_UI_ADD_FUNDS_QR},
         { "addFundsText", IDS_BRAVE_UI_ADD_FUNDS_TEXT},
         { "addFundsTitle", IDS_BRAVE_UI_ADD_FUNDS_TITLE},
+        { "adsEarnings", IDS_BRAVE_UI_ADS_EARNINGS },
         { "allowTip", IDS_BRAVE_UI_ALLOW_TIP },
         { "amount", IDS_BRAVE_UI_AMOUNT },
         { "backup", IDS_BRAVE_UI_BACKUP },

diff --git a/common/extensions/api/brave_rewards.json b/common/extensions/api/brave_rewards.json
@@ -177,6 +177,10 @@
                     "expiryTime": {
                       "type": "integer",
                       "description": "when grant expires, timestamp in seconds"
+                    },
+                    "type": {
+                      "type": "string",
+                      "description": "grant type, ugp or ads"
                     }
                   }
                 }

diff --git a/components/brave_rewards/browser/extension_rewards_service_observer.cc b/components/brave_rewards/browser/extension_rewards_service_observer.cc
@@ -80,6 +80,7 @@ void ExtensionRewardsServiceObserver::OnWalletProperties(
       grant.altcurrency = wallet_properties->grants[i].altcurrency;
       grant.probi = wallet_properties->grants[i].probi;
       grant.expiry_time = wallet_properties->grants[i].expiryTime;
+      grant.type = wallet_properties->grants[i].type;
     }
 
     std::unique_ptr<base::ListValue> args(

diff --git a/components/brave_rewards/browser/rewards_service_impl.cc b/components/brave_rewards/browser/rewards_service_impl.cc
@@ -804,6 +804,7 @@ void RewardsServiceImpl::OnWalletProperties(
         grant.altcurrency = wallet_info->grants_[i].altcurrency;
         grant.probi = wallet_info->grants_[i].probi;
         grant.expiryTime = wallet_info->grants_[i].expiryTime;
+        grant.type = wallet_info->grants_[i].type;
 
         wallet_properties->grants.push_back(grant);
       }
@@ -1431,6 +1432,7 @@ void RewardsServiceImpl::TriggerOnRecoverWallet(ledger::Result result,
     grant.altcurrency = grants[i].altcurrency;
     grant.probi = grants[i].probi;
     grant.expiryTime = grants[i].expiryTime;
+    grant.type = grants[i].type;
 
     newGrants.push_back(grant);
   }

diff --git a/components/brave_rewards/resources/extension/brave_rewards/_locales/en_US/messages.json b/components/brave_rewards/resources/extension/brave_rewards/_locales/en_US/messages.json
@@ -408,5 +408,9 @@
   "turnOnAds": {
     "message": "Turn on Ads",
     "description": "Prompt to turn on Ads via notification"
+  },
+  "adsEarnings": {
+    "message": "earned from ads",
+    "description": "Description text for ad grants in grant details"
   }
 }
diff --git a/components/brave_rewards/resources/extension/brave_rewards/background/api/locale_api.ts b/components/brave_rewards/resources/extension/brave_rewards/background/api/locale_api.ts
@@ -98,6 +98,7 @@ export const getUIMessages = (): Record<string, string> => {
     'unVerifiedText',
     'unVerifiedTextMore',
     'verifiedPublisher',
+    'adsEarnings',
     'welcomeButtonTextOne',
     'welcomeButtonTextTwo',
     'welcomeDescOne',

diff --git a/components/brave_rewards/resources/extension/brave_rewards/utils.ts b/components/brave_rewards/resources/extension/brave_rewards/utils.ts
@@ -39,14 +39,10 @@ export const getGrants = (grants?: RewardsExtension.Grant[]) => {
   }
 
   return grants.map((grant: RewardsExtension.Grant) => {
-    let expireDate = ''
-    if (grant.type !== 'ads') {
-      expireDate = new Date(grant.expiryTime * 1000).toLocaleDateString()
-    }
-
     return {
       tokens: convertProbiToFixed(grant.probi),
-      expireDate
+      expireDate: new Date(grant.expiryTime * 1000).toLocaleDateString(),
+      type: grant.type || 'ugp'
     }
   })
 }

diff --git a/components/brave_rewards/resources/ui/components/pageWallet.tsx b/components/brave_rewards/resources/ui/components/pageWallet.tsx
@@ -156,14 +156,10 @@ class PageWallet extends React.Component<Props, State> {
     }
 
     return grants.map((grant: Rewards.Grant) => {
-      let expireDate = ''
-      if (grant.type !== 'ads') {
-        expireDate = new Date(grant.expiryTime * 1000).toLocaleDateString()
-      }
-
       return {
         tokens: utils.convertProbiToFixed(grant.probi),
-        expireDate
+        expireDate: new Date(grant.expiryTime * 1000).toLocaleDateString(),
+        type: grant.type || 'ugp'
       }
     })
   }

diff --git a/components/resources/brave_components_strings.grd b/components/resources/brave_components_strings.grd
@@ -498,6 +498,7 @@
       <message name="IDS_BRAVE_UI_SITE_BANNER_NOTICE_NOTE" desc="">NOTE:</message>
       <message name="IDS_BRAVE_UI_SITE_BANNER_NOTICE_TEXT" desc="">This creator has not yet signed up to receive contributions from Brave users. Your browser will keep trying to contribute until they verify, or until 90 days have passed.</message>
       <message name="IDS_BRAVE_UI_SITE_UNVERIFIED_TEXT_MORE" desc="">Learn more.</message>
+      <message name="IDS_BRAVE_UI_ADS_EARNINGS" desc="Description text for ad grants in grant details">earned from ads</message>
 
       <!-- WebUI brave sync resources -->
       <!-- WebUI brave sync resources: Shared Content -->