From 8c9435d8ca046901fecad947d0d0dcd4dfc863a8 Mon Sep 17 00:00:00 2001
From: Juan <juandgoc@gmail.com>
Date: Thu, 26 Mar 2026 11:17:11 -0500
Subject: [PATCH] fix: send email on transcript page permissions fixed

---
 server/reflector/views/transcripts.py         | 8 ++------
 www/app/(app)/transcripts/shareAndPrivacy.tsx | 7 ++-----
 2 files changed, 4 insertions(+), 11 deletions(-)

diff --git a/server/reflector/views/transcripts.py b/server/reflector/views/transcripts.py
index c33b1cb0..ec5a854c 100644
--- a/server/reflector/views/transcripts.py
+++ b/server/reflector/views/transcripts.py
@@ -700,8 +700,6 @@ async def transcript_post_to_zulip(
     )
     if not transcript:
         raise HTTPException(status_code=404, detail="Transcript not found")
-    if not transcripts_controller.user_can_mutate(transcript, user_id):
-        raise HTTPException(status_code=403, detail="Not authorized")
     content = get_zulip_message(transcript, include_topics)
 
     message_updated = False
@@ -733,17 +731,15 @@ class SendEmailResponse(BaseModel):
 async def transcript_send_email(
     transcript_id: str,
     request: SendEmailRequest,
-    user: Annotated[auth.UserInfo, Depends(auth.current_user)],
+    user: Annotated[Optional[auth.UserInfo], Depends(auth.current_user_optional)],
 ):
     if not is_email_configured():
         raise HTTPException(status_code=400, detail="Email not configured")
-    user_id = user["sub"]
+    user_id = user["sub"] if user else None
     transcript = await transcripts_controller.get_by_id_for_http(
         transcript_id, user_id=user_id
     )
     if not transcript:
         raise HTTPException(status_code=404, detail="Transcript not found")
-    if not transcripts_controller.user_can_mutate(transcript, user_id):
-        raise HTTPException(status_code=403, detail="Not authorized")
     sent = await send_transcript_email([request.email], transcript)
     return SendEmailResponse(sent=sent)
diff --git a/www/app/(app)/transcripts/shareAndPrivacy.tsx b/www/app/(app)/transcripts/shareAndPrivacy.tsx
index f0e85e42..446f8795 100644
--- a/www/app/(app)/transcripts/shareAndPrivacy.tsx
+++ b/www/app/(app)/transcripts/shareAndPrivacy.tsx
@@ -178,14 +178,11 @@ export default function ShareAndPrivacy(props: ShareAndPrivacyProps) {
                   <ShareZulip
                     transcript={props.transcript}
                     topics={props.topics}
-                    disabled={toShareMode(shareMode.value) === "private"}
+                    disabled={toShareMode(shareMode.value) === "public"}
                   />
                 )}
                 {emailEnabled && (
-                  <ShareEmail
-                    transcript={props.transcript}
-                    disabled={toShareMode(shareMode.value) === "private"}
-                  />
+                  <ShareEmail transcript={props.transcript} disabled={false} />
                 )}
                 <ShareCopy
                   finalSummaryElement={props.finalSummaryElement}