Skip to content

Conversation

@kurotyan514
Copy link

  • add mail_service_spec.rb
  • set default from in SendMailMailer

- add `mail_service_spec.rb` 
- set default from in SendMailMailer
@grassiricardo
Copy link
Member

@kurotyan514 This PR failed in continuous integration...

@kurotyan514
Copy link
Author

@grassiricardo
it's kind of weird , i run rspec spec on local it's successful
do you have any ideal?
2017-10-20 1 09 16

@grassiricardo
Copy link
Member

@kurotyan514 At night, I'll deal with the problem.

@kurotyan514
Copy link
Author

@grassiricardo thanks!

@diogopms
Copy link
Contributor

@grassiricardo, @kurotyan514 probably you need to do a rebase with master branch
Thanks 👍

@grassiricardo
Copy link
Member

grassiricardo commented Oct 24, 2017

@kurotyan514 your test code is in error, see then image below:

screen shot 2017-10-23 at 23 39 23

does the pull did the merge with the master

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants