Merge pull request maestrano#202 from fgourichon/mnoe-464

[MNOE-464] - Remove connect button for non admin users
aluqueGH · Jun 6, 2017 · ea33178 · ea33178
2 parents 9065ccb + d801534
commit ea33178
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 1 deletion.
diff --git a/src/app/components/mnoe-api/organizations.svc.coffee b/src/app/components/mnoe-api/organizations.svc.coffee
@@ -230,4 +230,8 @@ angular.module 'mnoEnterpriseAngular'
       organizationSettings: (obj = null) -> _self.can.create.organizationSettings(obj) # call similar permission
     }
 
+    _self.can.connect = {
+      appInstance: -> _self.role.atLeastAdmin()
+    }
+
     return @
diff --git a/src/app/views/apps/dashboard-apps-list.controller.coffee b/src/app/views/apps/dashboard-apps-list.controller.coffee
@@ -26,6 +26,9 @@ angular.module 'mnoEnterpriseAngular'
       $scope.helper.displayCogwheel = ->
         MnoeOrganizations.can.update.appInstance()
 
+      $scope.helper.canConnectApp = ->
+        MnoeOrganizations.can.connect.appInstance()
+
       $scope.helper.canRenameApp = ->
         MnoeOrganizations.can.update.appInstance()
 

diff --git a/src/app/views/apps/dashboard-apps-list.html b/src/app/views/apps/dashboard-apps-list.html
@@ -60,7 +60,7 @@ <h2 class="section-title">
               </div>
 
               <!-- OAuth Connect Button -->
-              <div ng-show="helper.isOauthConnectBtnShown(app)">
+              <div ng-show="helper.isOauthConnectBtnShown(app) && helper.canConnectApp()">
                 <a class="btn btn-warning" href="" ng-click="helper.oAuthConnectPath(app)">
                   {{ 'mno_enterprise.templates.dashboard.apps_list.connect' | translate }}
                 </a>