kenobi/jellyfin-discord-bot
2
1
Fork 0
Code Issues 17 Pull Requests 1 Actions Packages Projects 1 Releases Wiki Activity

WIP: feat/20-reactions-for-roles #59

Draft
kenobi wants to merge 16 commits from feat/20-reactions-for-roles into master
pull from: feat/20-reactions-for-roles
merge into: kenobi:master
Conversation 0 Commits 16 Files Changed 7 +2 -2
2 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 88061c361c - Show all commits

2
server/helper/role.controller.ts
View File

@ -38,7 +38,7 @@ export default class RoleController {
.catch(error => { logger.error(`Could not find announcement_role with id ${config.bot.announcement_role}. Error: ${error}`, { requestId, guildId: guild.id }) })
}
public async manageAnnouncementRoles(guild: Guild, reaction: MessageReaction, requestId: string) {
public async assignAnnouncementRolesFromReactions(guild: Guild, reaction: MessageReaction, requestId: string) {
const guildId = guild.id
logger.info("Managing roles", { guildId, requestId })
magnetotail commented 2023-11-24 15:47:36 +01:00
Outdated
Review
Copy Link

method called reactions but parameter is singular

method called reactions but parameter is singular
kenobi commented 2023-11-25 19:51:46 +01:00
Outdated
Review
Copy Link

1f372b0aac

1f372b0aacd94d6eb5cfbe2e72f61dfdf77c1a31

2
server/structures/client.ts
View File

@ -176,7 +176,7 @@ export class ExtendedClient extends Client {
const reactions = message.reactions.resolve("🎫")
//logger.debug(`reactions: ${JSON.stringify(reactions, null, 2)}`, { guildId: guild.id, requestId })
if (reactions) {
this.roleController.manageAnnouncementRoles(message.guild, reactions, requestId)
this.roleController.assignAnnouncementRolesFromReactions(message.guild, reactions, requestId)
} else {
logger.error("Did not get reactions! Aborting!", { guildId: guild.id, requestId })
}