Remove isAdmin, isViewer from API response (#6738)

* Remove isAdmin,isViewer from API response

* snap

server/presenters/__snapshots__/user.test.ts.snap

@@ -6,9 +6,7 @@ exports[`presents a user 1`] = `
   "color": "#FF5C80",
   "createdAt": undefined,
   "id": "123",
-  "isAdmin": false,
   "isSuspended": false,
-  "isViewer": false,
   "lastActiveAt": undefined,
   "name": "Test User",
   "role": "member",
@@ -22,9 +20,7 @@ exports[`presents a user without slack data 1`] = `
   "color": "#FF5C80",
   "createdAt": undefined,
   "id": "123",
-  "isAdmin": false,
   "isSuspended": false,
-  "isViewer": false,
   "lastActiveAt": undefined,
   "name": "Test User",
   "role": "member",

server/presenters/user.ts

@@ -15,9 +15,7 @@ type UserPresentation = {
   lastActiveAt: Date | null;
   color: string;
   role: UserRole;
-  isAdmin: boolean;
   isSuspended: boolean;
-  isViewer: boolean;
   email?: string | null;
   language?: string;
   preferences?: UserPreferences | null;
@@ -34,9 +32,7 @@ export default function presentUser(
     avatarUrl: user.avatarUrl,
     color: user.color,
     role: user.role,
-    isAdmin: user.isAdmin,
     isSuspended: user.isSuspended,
-    isViewer: user.isViewer,
     createdAt: user.createdAt,
     updatedAt: user.updatedAt,
     lastActiveAt: user.lastActiveAt,

server/routes/api/users/users.test.ts

@@ -347,8 +347,7 @@ describe("#users.invite", () => {
     const body = await res.json();
     expect(res.status).toEqual(200);
     expect(body.data.sent.length).toEqual(1);
-    expect(body.data.users[0].isAdmin).toBeTruthy();
-    expect(body.data.users[0].isViewer).toBeFalsy();
+    expect(body.data.users[0].role).toEqual(UserRole.Admin);
   });
 
   it("should invite user as a viewer", async () => {