From a8e83ccaa73fad83626a05646dc7daeafdcb15e4 Mon Sep 17 00:00:00 2001
From: Jazz Turner-Baggs <473256+jazzz@users.noreply.github.com>
Date: Thu, 30 Apr 2026 07:11:04 -0700
Subject: [PATCH] PR review changes

---
 core/conversations/src/conversation/group_v1.rs | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/core/conversations/src/conversation/group_v1.rs b/core/conversations/src/conversation/group_v1.rs
index b7d8c87..ff2629e 100644
--- a/core/conversations/src/conversation/group_v1.rs
+++ b/core/conversations/src/conversation/group_v1.rs
@@ -145,11 +145,9 @@ where
         convo_id: String,
         group_id: GroupId,
     ) -> Result<Self, ChatError> {
-        let Some(mls_group) = MlsGroup::load(ctx.borrow().provider().storage(), &group_id)
+        let mls_group = MlsGroup::load(ctx.borrow().provider().storage(), &group_id)
             .map_err(ChatError::generic)?
-        else {
-            return Err(ChatError::NoConvo("mls group not found".into()));
-        };
+            .ok_or_else(|| ChatError::NoConvo("mls group not found".into()))?;
 
         Self::subscribe(&mut *ds.borrow_mut(), &convo_id)?;