SUP-1317 add / hide skills

Author: Parth Shah

app/directives/skill-tile/skill-tile.directive.jade

@@ -1,4 +1,7 @@
+a(ng-click="enableHide && toggle()")
+
+  i.colored(ng-show="icon", ng-class="icon") 
+  i.default(ng-hide="icon") {}
+  .name {{skill.tagName}} [{{skill.score}}]
+  spam(ng-if="skill.hidden") Hidden
 
-i.colored(ng-show="icon", ng-class="icon") 
-i.default(ng-hide="icon") {}
-.name {{skill.tagName}}

app/directives/skill-tile/skill-tile.directive.js

@@ -38,16 +38,34 @@
       restrict: 'E',
       templateUrl: 'directives/skill-tile/skill-tile.directive.html',
       scope: {
-        skill: '=skill'
+        skill: '=',
+        enableHide: "="
       },
-      controller: ['$scope', function($scope) {
+      controller: ['$scope', 'ProfileService', 'UserService', function($scope, ProfileService, UserService) {
 
         for (var key in _iconMap) {
           if ($scope.skill.tagName.trim().toLowerCase().indexOf(key) > -1) {
             $scope.icon = _iconMap[key];
             break;
           }
         }
+
+        $scope.toggle = function() {
+          var skillTagId = $scope.skill.tagId;
+          var handle = UserService.getUserIdentity().handle;
+          if ($scope.skill.hidden) {
+            // un-hide skill
+            ProfileService.addUserSkill(handle, skillTagId).then(function(_skills) {
+              $scope.skill.hidden = false;
+            });
+          } else {
+            // hide skill
+            ProfileService.hideUserSkill(handle, skillTagId).then(function(_skills) {
+              $scope.skill.hidden = true;
+            });
+          }
+        };
+
       }]
     };
   });

app/index.jade

@@ -90,7 +90,7 @@ html
     script(src='../bower_components/angucomplete-alt/angucomplete-alt.js')
     script(src='../bower_components/angular-cookies/angular-cookies.js')
     script(src='../bower_components/angular-dropdowns/dist/angular-dropdowns.js')
-    script(src='../bower_components/angular-filter/dist/angular-filter.min.js')
+    script(src='../bower_components/angular-filter/dist/angular-filter.js')
     script(src='../bower_components/angular-img-fallback/angular.dcb-img-fallback.js')
     script(src='../bower_components/angular-jwt/dist/angular-jwt.js')
     script(src='../bower_components/angular-sanitize/angular-sanitize.js')

app/services/api.service.js

@@ -102,6 +102,16 @@
               return null; // data
             }
           })
+          .addElementTransformer('skills', function(elem) {
+            // transform map to simple array
+            var skills = []
+            _.forEach(elem.skills, function(n,k) {
+              n.tagId = k;
+              skills.push(n);
+            });
+            elem.skills = skills;
+            return elem;
+          })
           .setErrorInterceptor(function(response) {
             // TODO
             switch (response.status) {

app/services/profile.service.js

@@ -15,6 +15,7 @@
       updateUserProfile: updateUserProfile,
       getUserSkills: getUserSkills,
       addUserSkill: addUserSkill,
+      hideUserSkill: hideUserSkill,
       getUserFinancials: getUserFinancials,
       getUserStats: getUserStats,
       getDistributionStats: getDistributionStats,
@@ -42,11 +43,16 @@
     function getUserSkills(username) {
       return restangular.one('members', username).one('skills').get();
     }
-    
+
     function addUserSkill(username, skillTagId) {
-      var body = {};
-      body[skillTagId] = { 'hidden': false };
-      return restangular.one('members', username).one('skills').patch(JSON.stringify(body));
+      var body = { skills:{} };
+      body['skills'][skillTagId] = { 'hidden': false };
+      return restangular.one('members', username).one('skills').patch(body);
+    }
+    function hideUserSkill(username, skillTagId) {
+      var body = { skills:{} };
+      body['skills'][skillTagId] = { 'hidden': true };
+      return restangular.one('members', username).one('skills').patch(body);
     }
 
     function getUserFinancials(username) {

app/services/tags.service.js

@@ -3,14 +3,16 @@
 
   angular.module('tc.services').factory('TagsService', TagsService);
 
-  TagsService.$inject = ['ApiService'];
+  TagsService.$inject = ['ApiService', '$q'];
 
-  function TagsService(ApiService) {
+  function TagsService(ApiService, $q) {
     var api = ApiService.restangularV3;
-    
+
     var service = {
       getApprovedSkillTags: function() {
-        return api.one("tags").get({filter: 'domain=SKILLS&status=APPROVED'});
+        return api.all("tags")
+          .withHttpConfig({skipAuthorization: true})
+          .getList({filter: 'domain=SKILLS&status=APPROVED'});
       }
     };
 

app/settings/edit-profile/edit-profile.controller.js

@@ -23,7 +23,7 @@
 
       processData(userData);
       vm.userData = userData;
-      
+
       TagsService.getApprovedSkillTags()
       .then(function(tags) {
         vm.tags = tags;
@@ -40,10 +40,17 @@
         $log.error(err);
       });
     }
-    
+
     function addSkill(skill) {
-      var skillTagId = _.get(skill, 'originalObject.id');
-      ProfileService.addUserSkill(vm.userData.handle, skillTagId);
+      if (skill) {
+        var skillTagId = _.get(skill, 'originalObject.id').toString();
+        ProfileService.addUserSkill(vm.userData.handle, skillTagId).then(function(resp) {
+          // find the new skill in response object and inject it into our existing list.
+          // we dont want to replace the entire object / map  because we will lose hidden tags
+          var newSkill = _.find(resp.skills, {tagId: skillTagId});
+          vm.skills.push(newSkill);
+        });
+      }
     }
 
     function updateCountry(angucompleteCountryObj) {

app/settings/edit-profile/edit-profile.jade

@@ -104,8 +104,8 @@
     )
 
     .skills
-      .skill(ng-repeat="skill in vm.skills")
-        skill-tile(skill="skill")
+      .skill(ng-repeat="skill in vm.skills | orderBy:'-score'")
+        skill-tile(skill="skill" enable-hide="true")
 
   .clearfix