Correct permissions (#759)

all-of-us · Apr 25, 2018 · 4865d73 · 4865d73
1 parent 61cc751
commit 4865d73
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 2 deletions.
diff --git a/ui/src/app/views/workspace-nav-bar/component.html b/ui/src/app/views/workspace-nav-bar/component.html
@@ -8,7 +8,7 @@
     <button class="btn btn-primary btn-top">Notebooks</button>
   -->
   <span role="tooltip" aria-haspopup="true" class="tooltip tooltip-md tooltip-top-right">
-    <clr-button class="btn btn-primary btn-top" [class.awaiting-review]="awaitingReview" [disabled]="awaitingReview"
+    <clr-button class="btn btn-primary btn-top" [class.awaiting-review]="awaitingReview" [disabled]="awaitingReview || !writePermission"
             [routerLink]="['/workspace', wsNamespace, wsId, 'cohorts', 'build']">
       Cohort Builder
     </clr-button>

diff --git a/ui/src/app/views/workspace/component.html b/ui/src/app/views/workspace/component.html
@@ -24,7 +24,7 @@ <h2>{{workspace.name}}</h2>
       <div>
         <span role="tooltip" aria-haspopup="true" class="tooltip tooltip-md tooltip-top-left">
           <button *ngIf="tabOpen===Tabs.Cohorts" class="btn btn-primary btn-create"
-                  [class.awaiting-review]="awaitingReview" [disabled]="awaitingReview" (click)="buildCohort()">
+                  [class.awaiting-review]="awaitingReview" [disabled]="awaitingReview || !writePermission" (click)="buildCohort()">
             + Add a Cohort
           </button>
           <span *ngIf="awaitingReview" class="tooltip-content">Workspace is awaiting review</span>

diff --git a/ui/src/app/views/workspace/component.ts b/ui/src/app/views/workspace/component.ts
@@ -369,4 +369,9 @@ export class WorkspaceComponent implements OnInit, OnDestroy {
     const asDate = new Date(this.workspace.lastModifiedTime);
     return asDate.toDateString();
   }
+
+  get writePermission(): boolean {
+    return this.accessLevel === WorkspaceAccessLevel.OWNER
+      || this.accessLevel === WorkspaceAccessLevel.WRITER;
+  }
 }