Skip to content
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

thread_auto_close timer starts after user replied or after note is added #3030

Closed
Flybel opened this issue May 13, 2021 · 1 comment
Closed
Labels
code improvement maybe: bug An unconfirmed bug staged Staged for next version

Comments

@Flybel
Copy link

Flybel commented May 13, 2021

In the config wiki, thread_auto_close is described as:

Time that has to pass after no reply from a user before the thread auto closes. If not set, thread will not auto close.

However, this seems misleading as the timer begins whenever a message is sent in a thread by anyone or when a note is added. This also produces undesirable behaviour. Examples:

  • A user contacts ModMail to request assistance. The ticket will be closed even if no reply is given at all.
  • A staff member replies and issues a ?close cancel to put the issue on hold. While staff member is unavailable, the user replies with additional info, which resets the timer and the ticket is closed before staff member gets back.
  • A staff member issues a ?close cancel and then a ?note. The timer is restarted and the ticket is closed.

Possible changes for this can be to completely disable the timer for anything other than a ?reply being issued by a staff member, or to add a second timer configuration for ?reply versus messages sent by the user in DMs. The ?note command should not be considered for this in both cases.

@fourjr fourjr added code improvement maybe: bug An unconfirmed bug labels Jul 4, 2021
@fourjr
Copy link
Collaborator

fourjr commented Aug 7, 2021

This issue will be resolved in v3.10.0-dev5

Changes:

  • Notes no longer restart the timer.
  • The timer only restarts when the reply is sent "from mod".

@fourjr fourjr added the staged Staged for next version label Aug 7, 2021
@fourjr fourjr closed this as completed Sep 4, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
code improvement maybe: bug An unconfirmed bug staged Staged for next version
Projects
None yet
Development

No branches or pull requests

2 participants