Infrastructure/ghost
0
Fork 0
mirror of https://github.com/TryGhost/Ghost.git synced 2025-01-06 22:40:14 -05:00
Code Issues Activity

Sends test email to the person who clicked on the button

Browse source 
Closes #3649

* Small refactor of the api route for email test. We no longer need to look up the email in the settings
* Added a user model lookup based on context to the test email method.
This commit is contained in:
Gabor Javorszky 2014-08-08 18:41:14 +01:00
parent 9eb2789d68
commit 508f669a49
2 changed files with 17 additions and 21 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
core/server/api/mail.js
View file

@ -58,20 +58,24 @@ mail = {
* @param {Object} required property 'to' which contains the recipient address
* @returns {Promise}
*/
sendTest: function (object, options) {
sendTest: function (options) {
var user = require('../models/user').User;
return user.findOne({id: options.context.user}).then(function (result) {
return mail.generateContent({template: 'test'}).then(function (emailContent) {
var payload = {mail: [{
message: {
to: object.to,
to: result.get('email'),
subject: 'Test Ghost Email',
html: emailContent.html,
text: emailContent.text
}
}]};
return mail.send(payload, options);
});
}, function () {
return when.reject(new errors.NotFoundError('Could not find the current user'));
});
},
/**

8
core/server/routes/api.js
View file

@ -58,15 +58,7 @@ apiRoutes = function (middleware) {
// ## Mail
router.post('/mail', api.http(api.mail.send));
router.post('/mail/test', function (req, res) {
api.settings.read('email').then(function (result) {
// attach the to: address to the request body so that it is available
// to the http api handler
req.body = { to: result.settings[0].value };
api.http(api.mail.sendTest)(req, res);
}).catch(function () {
api.http(api.mail.sendTest)(req, res);
});
});