Don't throw error when no client (#811)

xmtp · Feb 1, 2025 · fd43af4 · fd43af4
1 parent 5af4ed8
commit fd43af4
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 15 deletions.
diff --git a/apps/xmtp.chat/src/hooks/useConversation.ts b/apps/xmtp.chat/src/hooks/useConversation.ts
@@ -4,7 +4,6 @@ import type {
   SafeListMessagesOptions,
 } from "@xmtp/browser-sdk";
 import { useState } from "react";
-import { ClientNotFoundError } from "../helpers/errors";
 import { useClient } from "./useClient";
 
 export const useConversation = (conversation?: Conversation) => {
@@ -19,7 +18,7 @@ export const useConversation = (conversation?: Conversation) => {
     syncFromNetwork: boolean = false,
   ) => {
     if (!client) {
-      throw new ClientNotFoundError("fetching messages");
+      return;
     }
 
     if (syncFromNetwork) {
@@ -39,7 +38,7 @@ export const useConversation = (conversation?: Conversation) => {
 
   const sync = async () => {
     if (!client) {
-      throw new ClientNotFoundError("syncing messages");
+      return;
     }
 
     setSyncing(true);
@@ -53,7 +52,7 @@ export const useConversation = (conversation?: Conversation) => {
 
   const send = async (message: string) => {
     if (!client) {
-      throw new ClientNotFoundError("sending message");
+      return;
     }
 
     setSending(true);

diff --git a/apps/xmtp.chat/src/hooks/useConversations.ts b/apps/xmtp.chat/src/hooks/useConversations.ts
@@ -3,7 +3,6 @@ import type {
   SafeListConversationsOptions,
 } from "@xmtp/browser-sdk";
 import { useState } from "react";
-import { ClientNotFoundError } from "../helpers/errors";
 import { useClient } from "./useClient";
 
 export const useConversations = () => {
@@ -16,7 +15,7 @@ export const useConversations = () => {
     syncFromNetwork: boolean = false,
   ) => {
     if (!client) {
-      throw new ClientNotFoundError("fetching conversations");
+      return;
     }
 
     if (syncFromNetwork) {
@@ -35,7 +34,7 @@ export const useConversations = () => {
 
   const sync = async () => {
     if (!client) {
-      throw new ClientNotFoundError("syncing conversations");
+      return;
     }
 
     setSyncing(true);
@@ -49,7 +48,7 @@ export const useConversations = () => {
 
   const getConversationById = async (conversationId: string) => {
     if (!client) {
-      throw new ClientNotFoundError("fetching a conversation by ID");
+      return;
     }
 
     setLoading(true);
@@ -65,7 +64,7 @@ export const useConversations = () => {
 
   const getMessageById = async (messageId: string) => {
     if (!client) {
-      throw new ClientNotFoundError("fetching a message by ID");
+      return;
     }
 
     setLoading(true);
@@ -83,7 +82,7 @@ export const useConversations = () => {
     options: SafeCreateGroupOptions,
   ) => {
     if (!client) {
-      throw new ClientNotFoundError("creating a new group");
+      return;
     }
 
     setLoading(true);
@@ -101,7 +100,7 @@ export const useConversations = () => {
 
   const newDm = async (member: string) => {
     if (!client) {
-      throw new ClientNotFoundError("creating a new DM");
+      return;
     }
 
     setLoading(true);

diff --git a/apps/xmtp.chat/src/hooks/useIdentity.ts b/apps/xmtp.chat/src/hooks/useIdentity.ts
@@ -1,6 +1,5 @@
 import type { SafeInstallation } from "@xmtp/browser-sdk";
 import { useEffect, useState } from "react";
-import { ClientNotFoundError } from "../helpers/errors";
 import { useClient } from "./useClient";
 
 export const useIdentity = (syncOnMount: boolean = false) => {
@@ -20,7 +19,7 @@ export const useIdentity = (syncOnMount: boolean = false) => {
 
   const sync = async () => {
     if (!client) {
-      throw new ClientNotFoundError("syncing");
+      return;
     }
 
     setSyncing(true);
@@ -41,7 +40,7 @@ export const useIdentity = (syncOnMount: boolean = false) => {
 
   const revokeInstallation = async (installationIdBytes: Uint8Array) => {
     if (!client) {
-      throw new ClientNotFoundError("revoking an installation");
+      return;
     }
 
     setRevoking(true);
@@ -55,7 +54,7 @@ export const useIdentity = (syncOnMount: boolean = false) => {
 
   const revokeAllOtherInstallations = async () => {
     if (!client) {
-      throw new ClientNotFoundError("revoking all other installations");
+      return;
     }
 
     setRevoking(true);