Merge pull request #355 from appirio-tech/feature/hide-no-money

Feature/hide no money

Author: nlitwin

app/my-dashboard/header-dashboard/header-dashboard.controller.js

@@ -7,29 +7,41 @@
     '$stateParams',
     'NotificationService',
     'ProfileService',
-    'userIdentity'
+    'userIdentity',
+    '$log'
   ];
 
-  function HeaderDashboardController($stateParams, NotificationService, ProfileService, userIdentity) {
+  function HeaderDashboardController($stateParams, NotificationService, ProfileService, userIdentity, $log) {
     var vm = this;
-    vm.loading = true;
 
-    if ($stateParams.notifyReset) {
-      NotificationService.inform('Thanks. Your new password has been set.');
-    }
 
     activate();
 
     function activate() {
+      vm.loading = true;
+      vm.hideMoney = false;
       var handle = userIdentity.handle;
 
+      if ($stateParams.notifyReset) {
+        NotificationService.inform('Thanks. Your new password has been set.');
+      }
+
+      displayMoneyEarned(handle);
+    }
+
+    function displayMoneyEarned(handle) {
       ProfileService.getUserFinancials(handle)
       .then(function(financials) {
         vm.moneyEarned = _.sum(_.pluck(financials, 'amount'));
-        vm.loading = false;
 
+        if (!vm.moneyEarned) {
+          vm.hideMoney = true;
+        }
+
+        vm.loading = false;
       })
       .catch(function(err) {
+        vm.hideMoney = true;
         vm.loading = false;
       });
     }

app/my-dashboard/header-dashboard/header-dashboard.jade

@@ -2,7 +2,7 @@ header
   h1 My Dashboard
 
   .user-stats(id="metrics", ng-hide="vm.loading")
-    .money-earned
+    .money-earned(ng-hide="vm.hideMoney")
       p.number(ng-bind="vm.moneyEarned | currency:'$':0")
 
       p Earned