Make channel ID optional when replying
This commit is contained in:
parent
df9153f5ba
commit
d3ece937e6
6309
assets/schemas.json
6309
assets/schemas.json
File diff suppressed because it is too large
Load Diff
@ -1,17 +1,17 @@
|
|||||||
/*
|
/*
|
||||||
Spacebar: A FOSS re-implementation and extension of the Discord.com backend.
|
Spacebar: A FOSS re-implementation and extension of the Discord.com backend.
|
||||||
Copyright (C) 2023 Spacebar and Spacebar Contributors
|
Copyright (C) 2023 Spacebar and Spacebar Contributors
|
||||||
|
|
||||||
This program is free software: you can redistribute it and/or modify
|
This program is free software: you can redistribute it and/or modify
|
||||||
it under the terms of the GNU Affero General Public License as published
|
it under the terms of the GNU Affero General Public License as published
|
||||||
by the Free Software Foundation, either version 3 of the License, or
|
by the Free Software Foundation, either version 3 of the License, or
|
||||||
(at your option) any later version.
|
(at your option) any later version.
|
||||||
|
|
||||||
This program is distributed in the hope that it will be useful,
|
This program is distributed in the hope that it will be useful,
|
||||||
but WITHOUT ANY WARRANTY; without even the implied warranty of
|
but WITHOUT ANY WARRANTY; without even the implied warranty of
|
||||||
MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
|
MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
|
||||||
GNU Affero General Public License for more details.
|
GNU Affero General Public License for more details.
|
||||||
|
|
||||||
You should have received a copy of the GNU Affero General Public License
|
You should have received a copy of the GNU Affero General Public License
|
||||||
along with this program. If not, see <https://www.gnu.org/licenses/>.
|
along with this program. If not, see <https://www.gnu.org/licenses/>.
|
||||||
*/
|
*/
|
||||||
@ -91,11 +91,10 @@ router.patch(
|
|||||||
}
|
}
|
||||||
} else rights.hasThrow("SELF_EDIT_MESSAGES");
|
} else rights.hasThrow("SELF_EDIT_MESSAGES");
|
||||||
|
|
||||||
|
// @ts-expect-error Something is wrong with message_reference here, TS complains since "channel_id" is optional in MessageCreateSchema
|
||||||
const new_message = await handleMessage({
|
const new_message = await handleMessage({
|
||||||
...message,
|
...message,
|
||||||
// TODO: should message_reference be overridable?
|
// TODO: should message_reference be overridable?
|
||||||
// eslint-disable-next-line @typescript-eslint/ban-ts-comment
|
|
||||||
// @ts-ignore
|
|
||||||
message_reference: message.message_reference,
|
message_reference: message.message_reference,
|
||||||
...body,
|
...body,
|
||||||
author_id: message.author_id,
|
author_id: message.author_id,
|
||||||
|
@ -117,6 +117,12 @@ export async function handleMessage(opts: MessageOptions): Promise<Message> {
|
|||||||
const guild = await Guild.findOneOrFail({
|
const guild = await Guild.findOneOrFail({
|
||||||
where: { id: channel.guild_id },
|
where: { id: channel.guild_id },
|
||||||
});
|
});
|
||||||
|
|
||||||
|
if (!opts.message_reference.guild_id)
|
||||||
|
opts.message_reference.guild_id = channel.guild_id;
|
||||||
|
if (!opts.message_reference.channel_id)
|
||||||
|
opts.message_reference.channel_id = opts.channel_id;
|
||||||
|
|
||||||
if (!guild.features.includes("CROSS_CHANNEL_REPLIES")) {
|
if (!guild.features.includes("CROSS_CHANNEL_REPLIES")) {
|
||||||
if (opts.message_reference.guild_id !== channel.guild_id)
|
if (opts.message_reference.guild_id !== channel.guild_id)
|
||||||
throw new HTTPError(
|
throw new HTTPError(
|
||||||
@ -127,6 +133,8 @@ export async function handleMessage(opts: MessageOptions): Promise<Message> {
|
|||||||
"You can only reference messages from this channel",
|
"You can only reference messages from this channel",
|
||||||
);
|
);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
message.message_reference = opts.message_reference;
|
||||||
}
|
}
|
||||||
/** Q: should be checked if the referenced message exists? ANSWER: NO
|
/** Q: should be checked if the referenced message exists? ANSWER: NO
|
||||||
otherwise backfilling won't work **/
|
otherwise backfilling won't work **/
|
||||||
|
@ -42,7 +42,7 @@ export interface MessageCreateSchema {
|
|||||||
};
|
};
|
||||||
message_reference?: {
|
message_reference?: {
|
||||||
message_id: string;
|
message_id: string;
|
||||||
channel_id: string;
|
channel_id?: string;
|
||||||
guild_id?: string;
|
guild_id?: string;
|
||||||
fail_if_not_exists?: boolean;
|
fail_if_not_exists?: boolean;
|
||||||
};
|
};
|
||||||
|
Loading…
x
Reference in New Issue
Block a user