Skip to content

Added thread_creation_silent#2887

Closed
zoemartin01 wants to merge 2 commits into
modmail-dev:masterfrom
zoemartin01:development
Closed

Added thread_creation_silent#2887
zoemartin01 wants to merge 2 commits into
modmail-dev:masterfrom
zoemartin01:development

Conversation

@zoemartin01

Copy link
Copy Markdown
Contributor

I just added thread_auto_open_silently with the default value 'false' as this allows for the users customisation on a recently changed feature.

@Aloqeely

Copy link
Copy Markdown
Contributor

You forgot to add the config in core/config_help.json

… more consistent with the other names and added it to config_help.json
@zoemartin01 zoemartin01 changed the title Added thread_auto_open_silently Added thread_creation_silent Nov 16, 2020
Comment thread core/config_help.json
},
"thread_creation_silent": {
"default": "No",
"description": "Setting this configuration will open a new thread silently by default.",

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
"description": "Setting this configuration will open a new thread silently by default.",
"description": "Setting this configuration will always open a new thread silently in contact.",

@fourjr fourjr left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Config variable should be changed to thread_contact_silent as it only affects contact

Comment thread core/config.py
"thread_auto_close_response": "This thread has been closed automatically due to inactivity after {timeout}.",
"thread_creation_response": "The staff team will get back to you as soon as possible.",
"thread_creation_footer": "Your message has been sent",
"thread_creation_silent": False,

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it has to be added to booleans too

@fourjr

fourjr commented Nov 19, 2020

Copy link
Copy Markdown
Collaborator

Thanks for the contributions! This PR has been merged in a123bf6, not sure why the PR wasn't autoclosed.

@fourjr fourjr closed this Nov 19, 2020
@fourjr fourjr added the staged Staged for next version label Nov 19, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

staged Staged for next version

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants