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

Seeding Problem #44

Closed
sammy-kariuki opened this issue Jun 28, 2016 · 3 comments
Closed

Seeding Problem #44

sammy-kariuki opened this issue Jun 28, 2016 · 3 comments

Comments

@sammy-kariuki
Copy link

Hello,
I have been able to set up the project but I cannot seed. When I run the 'php artisan db:seed', I get the following error.

[Tylercd100\LERN\Exceptions\NotifierFailedException]
Expected response code 220 but got code "", with message ""

[Swift_TransportException]
Expected response code 220 but got code "", with message ""

Any help on this?

Best Regards,

Sammy.

@sroutier sroutier added the bug label Jun 28, 2016
@sroutier
Copy link
Collaborator

Hey @sammyhash,

Thanks for logging the issue. Found a couple of problems actually.
Please refresh your copy of the project and try again, let me know if you have any other problems.

FYI: The error that you are seeing is probably because your mail system is not properly configured, this combined with a couple coding mistakes and you are left with an error that make no sense.
I made a newbie mistake and had the password change code in the model fire off an notification email instead of putting that code in the controller. That is why when the seed command is invoked it tries to send an email notification.
Because of your issue, I changed a few things around

  1. Move the email notification code in the controller, that will prevent the seed command from crashing if the mail system is not configured.
  2. Added a new configuration setting to enable/disable email notifications. With a default to false, it allows the administrator to build and use the system before having the entire mail system configured.

Thanks again.
/S

@sammy-kariuki
Copy link
Author

I have been able to seed even without configuring the email system first.
Thank you very much for the quick response.

@sroutier
Copy link
Collaborator

You are Welcome.
Enjoy.

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