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

…ndus Done button made active on page dirty instead of only communities
appirio-tech · Oct 17, 2017 · 83f454d · 83f454d
2 parents e97535f + 6fb6527
commit 83f454d
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/app/skill-picker/skill-picker.controller.js b/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 --git a/app/skill-picker/skill-picker.jade b/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