Add scraper editor role

openaustralia · Sep 10, 2024 · e55e509 · e55e509
1 parent 62dc1e3
commit e55e509
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 10 deletions.
diff --git a/app/models/user.rb b/app/models/user.rb
@@ -39,7 +39,7 @@ def send_devise_notification(notification, *args)
 
   sig { returns(T::Boolean) }
   def can_login_to_admin?
-    has_role?(:admin) || has_role?(:api_editor)
+    has_role?(:admin) || has_role?(:api_editor) || has_role?(:scraper_editor)
   end
 
   # This is currently used when creating users via an alert

diff --git a/app/policies/admin/applications_policy.rb b/app/policies/admin/applications_policy.rb
@@ -7,12 +7,12 @@ class ApplicationsPolicy < ApplicationPolicy
 
     sig { returns(T::Boolean) }
     def index?
-      user.has_role?(:admin)
+      user.has_role?(:admin) || user.has_role?(:scraper_editor)
     end
 
     sig { returns(T::Boolean) }
     def show?
-      user.has_role?(:admin)
+      user.has_role?(:admin) || user.has_role?(:scraper_editor)
     end
 
     sig { returns(T::Boolean) }
@@ -23,7 +23,7 @@ def destroy?
     class Scope < ApplicationPolicy::Scope
       sig { returns(ActiveRecord::Relation) }
       def resolve
-        user.has_role?(:admin) ? scope.all : scope.none
+        user.has_role?(:admin) || user.has_role?(:scraper_editor) ? scope.all : scope.none
       end
     end
   end

diff --git a/app/policies/admin/authority_policy.rb b/app/policies/admin/authority_policy.rb
@@ -7,33 +7,33 @@ class AuthorityPolicy < ApplicationPolicy
 
     sig { returns(T::Boolean) }
     def create?
-      user.has_role?(:admin)
+      user.has_role?(:admin) || user.has_role?(:scraper_editor)
     end
 
     sig { returns(T::Boolean) }
     def index?
-      user.has_role?(:admin)
+      user.has_role?(:admin) || user.has_role?(:scraper_editor)
     end
 
     sig { returns(T::Boolean) }
     def show?
-      user.has_role?(:admin)
+      user.has_role?(:admin) || user.has_role?(:scraper_editor)
     end
 
     sig { returns(T::Boolean) }
     def update?
-      user.has_role?(:admin)
+      user.has_role?(:admin) || user.has_role?(:scraper_editor)
     end
 
     sig { returns(T::Boolean) }
     def import?
-      user.has_role?(:admin)
+      user.has_role?(:admin) || user.has_role?(:scraper_editor)
     end
 
     class Scope < ApplicationPolicy::Scope
       sig { returns(ActiveRecord::Relation) }
       def resolve
-        user.has_role?(:admin) ? scope.all : scope.none
+        user.has_role?(:admin) || user.has_role?(:scraper_editor) ? scope.all : scope.none
       end
     end
   end