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

FEATURE: Add site setting for disabling mailing list mode site wide #4059

Merged

Conversation

gdpelican
Copy link
Contributor

Playing around with the email settings some in anticipation of this work:
https://meta.discourse.org/t/more-granular-mailing-list-mode/38107

This change is discussed here:
https://meta.discourse.org/t/completely-disable-mailing-list-mode/37334/8

@discoursebot
Copy link

You've signed the CLA, gdpelican. Thank you! This pull request is ready for review.

@@ -18,6 +18,28 @@

end

describe ".mailing_list_mode" do
Copy link
Contributor

Choose a reason for hiding this comment

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

. is used for class methods. Since mailing_list_mode is an instance method, # should be used instead.

@gdpelican gdpelican force-pushed the feature/disable-mailing-list-mode branch from b8b315e to 2f00e21 Compare March 8, 2016 06:40
tgxworld added a commit that referenced this pull request Mar 8, 2016
…mode

FEATURE: Add site setting for disabling mailing list mode site wide
@tgxworld tgxworld merged commit ee8be3d into discourse:master Mar 8, 2016
@tgxworld
Copy link
Contributor

tgxworld commented Mar 8, 2016

Looks good. Thank you @gdpelican.

@gdpelican gdpelican deleted the feature/disable-mailing-list-mode branch March 23, 2016 06:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
4 participants