🔒 Allow moderators to manage communication templates (#2364)

* 🔒 Allow moderators to manage communication templates * ✅ Adjust test to check errors properly
bluesky-social · Apr 15, 2024 · 0f915a6 · 0f915a6
1 parent a2d5658
commit 0f915a6
Show file tree

Hide file tree

Showing 4 changed files with 11 additions and 11 deletions.
diff --git a/packages/ozone/src/api/communication/createTemplate.ts b/packages/ozone/src/api/communication/createTemplate.ts
@@ -10,9 +10,9 @@ export default function (server: Server, ctx: AppContext) {
       const db = ctx.db
       const { createdBy, ...template } = input.body
 
-      if (!access.isAdmin) {
+      if (!access.isModerator) {
         throw new AuthRequiredError(
-          'Must be an admin to create a communication template',
+          'Must be a moderator to create a communication template',
         )
       }
 

diff --git a/packages/ozone/src/api/communication/deleteTemplate.ts b/packages/ozone/src/api/communication/deleteTemplate.ts
@@ -10,9 +10,9 @@ export default function (server: Server, ctx: AppContext) {
       const db = ctx.db
       const { id } = input.body
 
-      if (!access.isAdmin) {
+      if (!access.isModerator) {
         throw new AuthRequiredError(
-          'Must be an admin to delete a communication template',
+          'Must be a moderator to delete a communication template',
         )
       }
 

diff --git a/packages/ozone/src/api/communication/updateTemplate.ts b/packages/ozone/src/api/communication/updateTemplate.ts
@@ -10,9 +10,9 @@ export default function (server: Server, ctx: AppContext) {
       const db = ctx.db
       const { id, updatedBy, ...template } = input.body
 
-      if (!access.isAdmin) {
+      if (!access.isModerator) {
         throw new AuthRequiredError(
-          'Must be an admin to update a communication template',
+          'Must be a moderator to update a communication template',
         )
       }
 

diff --git a/packages/ozone/tests/communication-templates.test.ts b/packages/ozone/tests/communication-templates.test.ts
@@ -42,11 +42,11 @@ describe('communication-templates', () => {
         { ...templateOne, createdBy: sc.dids.bob },
         {
           encoding: 'application/json',
-          headers: await network.ozone.modHeaders('moderator'),
+          headers: await network.ozone.modHeaders('triage'),
         },
       )
       await expect(moderatorReq).rejects.toThrow(
-        'Must be an admin to create a communication template',
+        'Must be a moderator to create a communication template',
       )
       const modReq = await agent.api.tools.ozone.communication.createTemplate(
         { ...templateOne, createdBy: sc.dids.bob },
@@ -105,19 +105,19 @@ describe('communication-templates', () => {
         { id: '1' },
         {
           encoding: 'application/json',
-          headers: await network.ozone.modHeaders('moderator'),
+          headers: await network.ozone.modHeaders('triage'),
         },
       )
 
       await expect(modReq).rejects.toThrow(
-        'Must be an admin to delete a communication template',
+        'Must be a moderator to delete a communication template',
       )
 
       await agent.api.tools.ozone.communication.deleteTemplate(
         { id: '1' },
         {
           encoding: 'application/json',
-          headers: await network.ozone.modHeaders('admin'),
+          headers: await network.ozone.modHeaders('moderator'),
         },
       )
       const list = await listTemplates()