Skip to content

Commit 764210d

Browse files
committed
add forwarding errors
1 parent 3aee305 commit 764210d

2 files changed

Lines changed: 11 additions & 5 deletions

File tree

.changeset/add-forward-errors.md

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
---
2+
default: patch
3+
---
4+
5+
Add detailed error messages to forwarding failures.

src/app/components/message/modals/MessageForward.tsx

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -109,7 +109,7 @@ export function MessageForwardInternal({
109109
const [isTargetSelected, setIsTargetSelected] = useState(false);
110110
const [isForwardSuccess, setIsForwardSuccess] = useState(false);
111111
const [isForwarding, setIsForwarding] = useState(false);
112-
const [isForwardError, setIsForwardError] = useState(false);
112+
const [forwardError, setForwardError] = useState<string | null>(null);
113113
const [targetRoomId, setTargetRoomId] = useState<string | null>(null);
114114

115115
// detect if it's a public room or not
@@ -285,15 +285,16 @@ export function MessageForwardInternal({
285285
setIsForwardSuccess(true);
286286
})
287287
.catch((err: unknown) => {
288+
const message = err instanceof Error ? err.message : String(err);
288289
debugLog.error('ui', 'Message forward failed', {
289290
sourceRoomId: room.roomId,
290291
targetRoomId: targetRoom.roomId,
291-
error: err instanceof Error ? err.message : String(err),
292+
error: message,
292293
});
293294
Sentry.metrics.count('sable.message.forward.error', 1);
294295
setIsForwarding(false);
295296
setIsForwardSuccess(false);
296-
setIsForwardError(true);
297+
setForwardError(message);
297298
});
298299
};
299300

@@ -348,9 +349,9 @@ export function MessageForwardInternal({
348349
<Text>Forward to {getRoom(targetRoomId)?.name}</Text>
349350
</Button>
350351
)}
351-
{isForwardError && (
352+
{forwardError && (
352353
<Text size="T300" color="Critical600" style={{ margin: config.space.S300 }}>
353-
Failed to forward message. Please try again.
354+
Failed to forward: {forwardError}
354355
</Text>
355356
)}
356357
</Box>

0 commit comments

Comments
 (0)