-
-
Notifications
You must be signed in to change notification settings - Fork 4.7k
Fixes thread_auto_close execution when disabled. #3423
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: development
Are you sure you want to change the base?
Conversation
This fixes the issue #3290 which caused threads to be auto-closed even if `thread_auto_close` has been disabled. There was also an issue that closed the thread when the user has responded to mods. The thread should stay open and only auto close when the staff has replied back.
|
Not sure why it merged them (the review requests) when I did that |
This solves the thread from autoclosing if the closure has been cancelled earlier in a thread.
sebkuip
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There's some extra mongo calls that I think could be removed to save bandwidth limits. Otherwise seems okay
I had added a small bugfix aswell for pagination when an invalid config var was given. This happened to occur upon removing the `thread_auto_close` config.
sebkuip
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
This fixes the issue #3290 which caused threads to be auto-closed even if
thread_auto_closehas been disabled.There was also an issue that closed the thread when the user has responded to mods. The thread should stay open and only auto close when the staff has replied back.