Dekescordes/spacebar
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

🐛 api member add roles -> make sure that roles exist

Browse Source
This commit is contained in:
Flam3rboy 2021-09-04 11:26:25 +02:00
parent 82341300fc
commit cc9f43fb63
1 changed files with 7 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
api/src/routes/guilds/#guild_id/members/#member_id/index.ts
View File

@ -38,14 +38,13 @@ router.patch("/", check(MemberChangeSchema), async (req: Request, res: Response)
member.roles = body.roles.map((x) => new Role({ id: x })); // foreign key constraint will fail if role doesn't exist member.roles = body.roles.map((x) => new Role({ id: x })); // foreign key constraint will fail if role doesn't exist
} }
Promise.all([ await member.save();
member.save(), // do not use promise.all as we have to first write to db before emitting the event
emitEvent({ await emitEvent({
event: "GUILD_MEMBER_UPDATE", event: "GUILD_MEMBER_UPDATE",
guild_id, guild_id,
data: { ...member, roles: member.roles.map((x) => x.id) } data: { ...member, roles: member.roles.map((x) => x.id) }
} as GuildMemberUpdateEvent) } as GuildMemberUpdateEvent);
]);
res.json(member); res.json(member);
}); });