Remove unused subscriptions RBAC role (#51717)

Not used at the moment, was a leftover in a PR that I forgot to remove.
sourcegraph · May 10, 2023 · 7cf2a50 · 7cf2a50
1 parent 7ba9b85
commit 7cf2a50
Show file tree

Hide file tree

Showing 5 changed files with 1 addition and 19 deletions.
diff --git a/client/web/src/rbac/constants.ts b/client/web/src/rbac/constants.ts
@@ -3,7 +3,3 @@
 export const BatchChangesReadPermission = 'BATCH_CHANGES#READ'
 
 export const BatchChangesWritePermission = 'BATCH_CHANGES#WRITE'
-
-export const SubscriptionsReadPermission = 'SUBSCRIPTIONS#READ'
-
-export const SubscriptionsWritePermission = 'SUBSCRIPTIONS#WRITE'
diff --git a/cmd/frontend/graphqlbackend/schema.graphql b/cmd/frontend/graphqlbackend/schema.graphql
@@ -9700,10 +9700,6 @@ enum PermissionNamespace {
     This represents the Batch Changes namespace.
     """
     BATCH_CHANGES
-    """
-    This represents the subscriptions namespace.
-    """
-    SUBSCRIPTIONS
 }
 
 """

diff --git a/internal/rbac/constants.go b/internal/rbac/constants.go
diff --git a/internal/rbac/schema.yaml b/internal/rbac/schema.yaml
@@ -3,7 +3,3 @@ namespaces:
     actions:
       - READ
       - WRITE
-  - name: SUBSCRIPTIONS
-    actions:
-      - READ
-      - WRITE
diff --git a/internal/types/types.go b/internal/types/types.go
@@ -874,8 +874,7 @@ func (n PermissionNamespace) String() string {
 // Valid checks if a namespace is valid and supported by the Sourcegraph RBAC system.
 func (n PermissionNamespace) Valid() bool {
 	switch n {
-	case BatchChangesNamespace,
-		SubscriptionsNamespace:
+	case BatchChangesNamespace:
 		return true
 	default:
 		return false
@@ -884,7 +883,6 @@ func (n PermissionNamespace) Valid() bool {
 
 // BatchChangesNamespace represents the Batch Changes namespace.
 const BatchChangesNamespace PermissionNamespace = "BATCH_CHANGES"
-const SubscriptionsNamespace PermissionNamespace = "SUBSCRIPTIONS"
 
 type Permission struct {
 	ID        int32