minor fixes

app/components/Notifications/NotificationListItem.tsx

@@ -36,7 +36,7 @@ function NotificationListItem({ notification, onNavigate }: Props) {
   const collection = collectionId ? collections.get(collectionId) : undefined;
   const [processing, setProcessing] = React.useState(false);
   const [selectedPermission, setSelectedPermission] =
-    React.useState<DocumentPermission>(DocumentPermission.ReadWrite);
+    React.useState<DocumentPermission>(DocumentPermission.Read);
 
   const isAccessRequest =
     notification.event === NotificationEventType.RequestDocumentAccess &&

server/routes/api/accessRequests/accessRequests.test.ts

@@ -304,7 +304,6 @@ describe("#accessRequests.approve", () => {
       teamId: team.id,
     });
 
-    // add non-manager to the document with editor access
     await UserMembership.create({
       userId: admin.id,
       documentId: document.id,
@@ -461,7 +460,6 @@ describe("#accessRequests.dismiss", () => {
       body: {
         token: admin.getJwtToken(),
         id: accessRequest.id,
-        permission: DocumentPermission.ReadWrite,
       },
     });
 

server/routes/api/accessRequests/schema.ts

@@ -35,7 +35,7 @@ export const AccessRequestsApproveSchema = BaseSchema.extend({
   body: BaseIdSchema.extend({
     permission: z
       .nativeEnum(DocumentPermission)
-      .default(DocumentPermission.ReadWrite),
+      .default(DocumentPermission.Read),
   }),
 });