Skip to content

Commit 08405e7

Browse files
committed
Improve UX by rendering delete button immediately
1 parent 5316bea commit 08405e7

File tree

1 file changed

+5
-1
lines changed

1 file changed

+5
-1
lines changed

app/commands/track.tsx

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ export const handler = async (
1616
) => {
1717
const { targetMessage: message, user } = interaction;
1818

19-
const { message: logMessage } = await reportUser({
19+
const reportPromise = reportUser({
2020
reason: ReportReasons.track,
2121
message,
2222
staff: user,
@@ -30,6 +30,10 @@ export const handler = async (
3030
label="Delete message"
3131
style="danger"
3232
onClick={async () => {
33+
// Need to ensure that we've finished reporting before we try to
34+
// respond to a click event.
35+
// Initiating at the top level and waiting here is a big UX win.
36+
const { message: logMessage } = await reportPromise;
3337
await Promise.allSettled([
3438
message.delete(),
3539
logMessage.reply({

0 commit comments

Comments
 (0)