cardinalitypuzzles / cardboard

Dashboard for managing puzzles and tracking status during a puzzle hunt
MIT License
31 stars 21 forks source link

When garbage collecting old Discord channels, remove the voice channel even if the text channel still exists #591

Closed rgossiaux closed 1 year ago

rgossiaux commented 2 years ago

Looks like if the text channel gets kept (because it contains text), then the voice channel also is kept. In this case we could keep the text channels and delete the voice channels.

rawxfish commented 1 year ago

The code's behavior is already like that from what I can tell. The only thing I can think of is the code bugs out when it tries to figure out if the text channel should be kept and the celery task never gets to the lines of code that cleans up voice.

Switched the order of the cleanups so audio happens before all the stuff happens for text in #668, which will hopefully resolve this issue.