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

Cannot disable the Mass Email #4490

Closed
brianteeman opened this issue Oct 9, 2014 · 6 comments
Closed

Cannot disable the Mass Email #4490

brianteeman opened this issue Oct 9, 2014 · 6 comments

Comments

@brianteeman
Copy link
Contributor

There is no way as far as I can see to disable Mass Email. The only ACL I can see is for the entire Users component

@zero-24
Copy link
Contributor

zero-24 commented Oct 16, 2014

@brianteeman any suggestion which level we should use? A new or a Default value?

@brianteeman
Copy link
Contributor Author

Well the problem is that we cant currently target the mass mail functionality separately to the rest of the users component. I am not bothered in how it is achieved just that it is possible

@dkanchev
Copy link

@brianteeman do you think that it is a good solution to add an option "Mass Mailer Enabled" Yes/No button to the System -> Global Configuration -> Server -> Mail Settings menu? If you think that this is a good solution we'll implement it.

This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/4490.

@brianteeman
Copy link
Contributor Author

Thats an option
On 17 Oct 2014 08:14, "dkanchev" notifications@github.com wrote:

@brianteeman https://github.com/brianteeman do you think that it is a
good solution to add an option "Mass Mailer Enabled" Yes/No button to the
System -> Global Configuration -> Server -> Mail Settings menu? If you
think that this is a good solution we'll implement it.

This comment was created with the J!Tracker Application
https://github.com/joomla/jissues at issues.joomla.org/joomla-cms/4490
http://issues.joomla.org/tracker/joomla-cms/4490.


Reply to this email directly or view it on GitHub
#4490 (comment).

@madasha
Copy link

madasha commented Oct 17, 2014

Created pull request for the suggested solution:

#4747

This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/4490.

@brianteeman
Copy link
Contributor Author

Awesome thanks. Closing here See #4747

This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/4490.

phproberto pushed a commit that referenced this issue Oct 23, 2014
…here necessary. Fixes #4490. Fixes #4747

Fixed for Hathor template as well

Turned the option around so that default functionality is Mass Mail enabled

Complying to coding standard

Complying to coding standard (had some spaces instead of tabs

Cleaning some whitespaces
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants