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

Added thread_creation_silent #2887

Closed
wants to merge 2 commits into from

Conversation

zoemartin01
Copy link
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
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
@@ -303,6 +303,17 @@
"See also: `thread_creation_title`, `thread_creation_response`, `thread_self_closable_creation_footer`, `thread_close_footer`."
]
},
"thread_creation_silent": {
"default": "No",
"description": "Setting this configuration will open a new thread silently by default.",
Copy link
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.",

Copy link
Collaborator

@fourjr fourjr left a comment

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

@@ -53,6 +53,7 @@ class ConfigManager:
"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
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
Copy link
Collaborator

fourjr commented Nov 19, 2020

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