Skip to content

Commit f211488

Browse files
committed
Quiet Slack slash command responses
1 parent 1cf43b9 commit f211488

2 files changed

Lines changed: 6 additions & 19 deletions

File tree

app/api/slack/commands/route.js

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -16,8 +16,6 @@ import {
1616
slackConnectUrl,
1717
slackHelpPayload,
1818
slackSavedDumpPayload,
19-
slackSavingPayload,
20-
slackRoomModalOpenedPayload,
2119
openSlackRoomModal,
2220
} from "../../../../src/server/slack";
2321
import { cleanString } from "../../../../src/server/validation";
@@ -42,7 +40,7 @@ export async function POST(request) {
4240
await postSlackResponse(responseUrl, ephemeralText(error.message || "couldn't open room setup."));
4341
}
4442
});
45-
return ok(slackRoomModalOpenedPayload());
43+
return ok({});
4644
}
4745

4846
after(async () => {
@@ -57,7 +55,7 @@ export async function POST(request) {
5755
}
5856
});
5957

60-
return ok(roomName !== null ? ephemeralText("creating a mumbl room...") : slackSavingPayload());
58+
return ok({});
6159
} catch (error) {
6260
return serverError(error);
6361
}
@@ -80,15 +78,15 @@ async function saveOrConnect({ teamId, slackUserId, content, sourceMeta }) {
8078
const existingConnection = await findSlackConnection({ teamId, slackUserId });
8179
if (existingConnection) {
8280
const dump = await saveSlackDump({ connection: existingConnection, content, sourceMeta });
83-
return slackSavedDumpPayload({ url: dumpUrl(dump.id) });
81+
return slackSavedDumpPayload({ url: dumpUrl(dump.id), compact: true });
8482
}
8583

8684
const email = await getSlackUserEmail({ teamId, slackUserId });
8785
const mumblUserId = await findMumblUserByEmail(email);
8886
if (mumblUserId) {
8987
const connection = await connectSlackUser({ teamId, slackUserId, mumblUserId });
9088
const dump = await saveSlackDump({ connection, content, sourceMeta });
91-
return slackSavedDumpPayload({ url: dumpUrl(dump.id) });
89+
return slackSavedDumpPayload({ url: dumpUrl(dump.id), compact: true });
9290
}
9391

9492
const pending = await createPendingSlackDump({ teamId, slackUserId, content, sourceMeta });

src/server/slack.js

Lines changed: 2 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -559,16 +559,9 @@ export function ephemeralText(text) {
559559
};
560560
}
561561

562-
export function slackSavingPayload() {
563-
return blockResponse({
564-
text: "saving to mumbl...",
565-
blocks: [
566-
section("*saving to mumbl...*\nkeeping it private while mumbl files it away."),
567-
],
568-
});
569-
}
562+
export function slackSavedDumpPayload({ url, shortcut = false, compact = false }) {
563+
if (compact) return ephemeralText("saved privately to mumbl.");
570564

571-
export function slackSavedDumpPayload({ url, shortcut = false }) {
572565
return blockResponse({
573566
text: shortcut ? "saved to mumbl privately." : "saved. only you can see this.",
574567
blocks: [
@@ -608,10 +601,6 @@ export function slackRoomNeedsNamePayload() {
608601
});
609602
}
610603

611-
export function slackRoomModalOpenedPayload() {
612-
return ephemeralText("opening room setup...");
613-
}
614-
615604
export function slackRoomCreatedPayload({ space, openUrl, roomUrl, teamReadsUrl }) {
616605
return blockResponse({
617606
text: `created a mumbl room for ${space.name}.`,

0 commit comments

Comments
 (0)