diff --git "a/src/api/routes/users/@me/guilds/\043guild_id/settings.ts" "b/src/api/routes/users/@me/guilds/\043guild_id/settings.ts" index 4b806cf..4538785 100644 --- "a/src/api/routes/users/@me/guilds/\043guild_id/settings.ts" +++ "b/src/api/routes/users/@me/guilds/\043guild_id/settings.ts" @@ -1,22 +1,14 @@ import { Router, Response, Request } from "express"; import { Channel, - ChannelOverride, Member, - UserGuildSettings, + OrmUtils, + UserGuildSettingsSchema, } from "@fosscord/util"; import { route } from "@fosscord/api"; const router = Router(); -// This sucks. I would use a DeepPartial, my own or typeorms, but they both generate inncorect schema -export interface UserGuildSettingsSchema - extends Partial> { - channel_overrides: { - [channel_id: string]: Partial; - }; -} - // GET doesn't exist on discord.com router.get("/", route({}), async (req: Request, res: Response) => { const user = await Member.findOneOrFail({ @@ -30,7 +22,7 @@ "/", route({ body: "UserGuildSettingsSchema" }), async (req: Request, res: Response) => { - const body = req.body as UserGuildSettings; + const body = req.body as UserGuildSettingsSchema; if (body.channel_overrides) { for (var channel in body.channel_overrides) { @@ -40,9 +32,11 @@ const user = await Member.findOneOrFail({ where: { id: req.user_id, guild_id: req.params.guild_id }, + select: ["settings"] + }); - user.settings = { ...user.settings, ...body }; - await user.save(); + OrmUtils.mergeDeep(user.settings || {}, body); + Member.update({ id: req.user_id, guild_id: req.params.guild_id }, user); res.json(user.settings); },