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

feat/40-reroll-on-disinterest #54

Merged
kenobi merged 73 commits from feat/40-reroll-on-disinterest into master 2023-11-19 20:24:36 +01:00
Conversation 2 Commits 73 Files Changed 23 +3 -2
1 changed files with 3 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit e61b3a7b16 - Show all commits

5
server/events/autoCreateVoteByWPEvent.ts
View File

@ -30,9 +30,10 @@ export async function execute(event: GuildScheduledEvent) {
logger.info("EVENT DOES NOT HAVE STARTDATE; CANCELLING", { guildId: event.guildId, requestId })
return
}
const sentMessage = await client.voteController.createVoteMessage(event, announcementChannel, movies, event.guild?.id ?? "", requestId)
const sentMessageText = client.voteController.createVoteMessageText(event.id, event.scheduledStartAt, movies, event.guild?.id ?? "", requestId)
magnetotail marked this conversation as resolved Outdated
magnetotail commented 2023-11-18 12:43:06 +01:00
Outdated
Review
Copy Link

Warum erst eine message erstellen lassen die dann vom gleichen controller in der nächsten Zeile verschickt wird? Fände besser dem Controller zu sagen "Schick ne Vote message, hier sind die Infos die du brauchst"

Warum erst eine message erstellen lassen die dann vom gleichen controller in der nächsten Zeile verschickt wird? Fände besser dem Controller zu sagen "Schick ne Vote message, hier sind die Infos die du brauchst"
kenobi commented 2023-11-18 17:30:49 +01:00
Outdated
Review
Copy Link

True. Habe das handling in den vote controller verlegt.
Die Input Parameter sind dadurch allerdings lang genug geworden, dass ich ein separates Interface dafür erstellt habe.
4600820889

True. Habe das handling in den vote controller verlegt. Die Input Parameter sind dadurch allerdings lang genug geworden, dass ich ein separates Interface dafür erstellt habe. 4600820889d046972bb264912f0ad929c8950dac
const sentMessage = await client.voteController.sendVoteMessage(sentMessageText, movies.length, announcementChannel)
sentMessage.pin() //todo: uncomment when bot has permission to pin messages. Also update closepoll.ts to only fetch pinned messages
sentMessage.pin()
magnetotail marked this conversation as resolved Outdated
magnetotail commented 2023-11-18 12:44:24 +01:00
Outdated
Review
Copy Link

Das pinnen sollte glaube ich auch nicht das event übernehmen sondern der Controller

Das pinnen sollte glaube ich auch nicht das event übernehmen sondern der Controller
magnetotail commented 2023-11-18 16:27:14 +01:00
Outdated
Review
Copy Link

Evtl halt über nen Parameter bestimmen. Auf lange Sicht gehört das eigentliche verschicken der Message eh in einen eigenen controller aber der voteController kann die Daten für eine voteMessage entgegennehmen die aufbereiten und dann über den messageController die eigentliche message schicken lassen

Evtl halt über nen Parameter bestimmen. Auf lange Sicht gehört das eigentliche verschicken der Message eh in einen eigenen controller aber der voteController kann die Daten für eine voteMessage entgegennehmen die aufbereiten und dann über den messageController die eigentliche message schicken lassen
kenobi commented 2023-11-18 17:31:31 +01:00
Outdated
Review
Copy Link

4600820889

4600820889d046972bb264912f0ad929c8950dac
kenobi commented 2023-11-18 18:26:01 +01:00
Outdated
Review
Copy Link

gelöst durch das Handling in #54 (comment)
Es gibt noch ein Auftreten vom 'separaten' Pinnen (vote.controller.ts:114). Dort ist ein bisschen mehr refactoring nötig um die Input Paramter zu füllen.
Dieser Kommentar sollte allerdings fertig behandelt sein.

gelöst durch das Handling in https://gitea.brudi.xyz/kenobi/jellyfin-discord-bot/pulls/54#issuecomment-526 Es gibt noch ein Auftreten vom 'separaten' Pinnen (vote.controller.ts:114). Dort ist ein bisschen mehr refactoring nötig um die Input Paramter zu füllen. Dieser Kommentar sollte allerdings fertig behandelt sein.
}
}