Merge pull request #1238 from appirio-tech/f2f-skill_picker-by_shubhendus

Done button made active on page dirty instead of only communities

Author: ajefts

Diff for: app/skill-picker/skill-picker.controller.js

@@ -196,7 +196,7 @@ import _ from 'lodash'
         }
         promises.push(ProfileService.updateUserSkills(vm.username, data))
       }
-      logger.debug('isCommunitiesDirty: ' + isCommunitiesDirty())
+
       if (isCommunitiesDirty()) {
         for(var communityName in vm.communities) {
           var community = vm.communities[communityName]

Diff for: app/skill-picker/skill-picker.jade

@@ -74,4 +74,4 @@
     type="button",
     tc-busy-button, tc-busy-when="vm.saving",
     ng-click="vm.submitSkills()",
-    ng-disabled="vm.disableDoneButton || !vm.isCommunitySelected()") Done
+    ng-disabled="vm.disableDoneButton || !vm.isPageDirty()") Done