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

Add a way to test email settings #2355

Closed
Spuds opened this issue Jan 21, 2016 · 5 comments
Closed

Add a way to test email settings #2355

Spuds opened this issue Jan 21, 2016 · 5 comments

Comments

@Spuds
Copy link
Contributor

Spuds commented Jan 21, 2016

I think there should be a link on the managemail page to send a test email to ensure you've set it up correctly. I'm kind of concerned that I might not have it setup correctly.

@emanuele45
Copy link
Contributor

I was looking at that one, though it's a bit tricky I think, more because of the expected behaviour.
What I was thinking to do was a link (along the lines of "clean hives") with (maybe) an AJAX call to send the email in background.
Though, I feel it would look odd: in this condition I would expect the setting I just entered (either saved or not) to be the ones used for the sending. But, using unsaved setting would be... confusing, because then it could lead to people not saving the settings (forgetful people like me for example 👼 ).

Could the solution be to save the settings while sending the test email? Informing the admin, of course.

@Spuds
Copy link
Contributor Author

Spuds commented Nov 6, 2016

Probably save on send ... actually I think we have the same condition on the Sphinx search settings .. if you try to connect to the search daemon without first hitting save, it will fail. This even though the fields are populated with some defaults, those defaults are not in the DB yet.

@emanuele45
Copy link
Contributor

I'm tempted to move it to 2.0

@Spuds
Copy link
Contributor Author

Spuds commented Dec 2, 2016

I think there are a few things that would have to be done .. new test template, testing all of the replacement fields, including pbe, etc etc ... moving it to 2.0 as I don't see us doing this

@Spuds Spuds modified the milestones: 2.0, 1.1 beta 4 Dec 2, 2016
@Spuds
Copy link
Contributor Author

Spuds commented Apr 14, 2024

Added a basic email form in the ACP that an admin can use to send ... pretty basic implementation for a start.

@Spuds Spuds closed this as completed in e57c6a4 Apr 14, 2024
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

2 participants