andela / pibbble

Dribbble Clone for Programmers
https://pibbble-staging.herokuapp.com/
13 stars 3 forks source link

Password reset fakes sending of email #65

Closed unicodeveloper closed 8 years ago

unicodeveloper commented 8 years ago

I try to do a password reset online...but I actually don't get any reset email

ghost commented 8 years ago

That should be because you are not a registered user. Ope's login feature has not been merged in.

Oladipupo Isola | Software Developer, Andela See what TechCrunch http://techcrunch.com/2015/06/25/backed-by-spark-capital-andela-will-develop-a-continent-of-tech-talent/ and StackOverflow http://blog.stackoverflow.com/2015/05/stack-overflow-and-andela-partner-to-provide-education-beyond-borders/ say about us! (c) +234-703-302-0216 | Andela.com http://andela.com/

On Tue, Oct 20, 2015 at 9:42 PM, Otemuyiwa Prosper <notifications@github.com

wrote:

Assigned #65 https://github.com/andela/pibbble/issues/65 to @andela-oisola https://github.com/andela-oisola.

— Reply to this email directly or view it on GitHub https://github.com/andela/pibbble/issues/65#event-440810266.

ghost commented 8 years ago

Oh. I just noticed it now. I think I have to remove the .env from the .gitignore. Or is there a better way?

Oladipupo Isola | Software Developer, Andela See what TechCrunch http://techcrunch.com/2015/06/25/backed-by-spark-capital-andela-will-develop-a-continent-of-tech-talent/ and StackOverflow http://blog.stackoverflow.com/2015/05/stack-overflow-and-andela-partner-to-provide-education-beyond-borders/ say about us! (c) +234-703-302-0216 | Andela.com http://andela.com/

On Tue, Oct 20, 2015 at 9:53 PM, Dipo Isola oladipupo.isola@andela.com wrote:

That should be because you are not a registered user. Ope's login feature has not been merged in.

Oladipupo Isola | Software Developer, Andela See what TechCrunch http://techcrunch.com/2015/06/25/backed-by-spark-capital-andela-will-develop-a-continent-of-tech-talent/ and StackOverflow http://blog.stackoverflow.com/2015/05/stack-overflow-and-andela-partner-to-provide-education-beyond-borders/ say about us! (c) +234-703-302-0216 | Andela.com http://andela.com/

On Tue, Oct 20, 2015 at 9:42 PM, Otemuyiwa Prosper < notifications@github.com> wrote:

Assigned #65 https://github.com/andela/pibbble/issues/65 to @andela-oisola https://github.com/andela-oisola.

— Reply to this email directly or view it on GitHub https://github.com/andela/pibbble/issues/65#event-440810266.

unicodeveloper commented 8 years ago

I have merged everyone's feature in and everything works. I have also registered but when I try to do a password reset I don't get an email

ghost commented 8 years ago

It may be because of the SendGrid settings in the .env.

Oladipupo Isola | Software Developer, Andela See what TechCrunch http://techcrunch.com/2015/06/25/backed-by-spark-capital-andela-will-develop-a-continent-of-tech-talent/ and StackOverflow http://blog.stackoverflow.com/2015/05/stack-overflow-and-andela-partner-to-provide-education-beyond-borders/ say about us! (c) +234-703-302-0216 | Andela.com http://andela.com/

On Tue, Oct 20, 2015 at 9:57 PM, Otemuyiwa Prosper <notifications@github.com

wrote:

I have merged everyone's feature in and everything works. I have also registered but when I try to do a password reset I don't get an email

— Reply to this email directly or view it on GitHub https://github.com/andela/pibbble/issues/65#issuecomment-149700724.

unicodeveloper commented 8 years ago

Shed more light on that

ghost commented 8 years ago

Details like MAIL_DRIVER, MAIL_HOST, MAIL_USERNAME, MAIL_PASSWORD & MAIL_PORT are in my .env file. I think that might be what's stopping it from sending because it sends on my system.

Oladipupo Isola | Software Developer, Andela See what TechCrunch http://techcrunch.com/2015/06/25/backed-by-spark-capital-andela-will-develop-a-continent-of-tech-talent/ and StackOverflow http://blog.stackoverflow.com/2015/05/stack-overflow-and-andela-partner-to-provide-education-beyond-borders/ say about us! (c) +234-703-302-0216 | Andela.com http://andela.com/

On Tue, Oct 20, 2015 at 9:59 PM, Otemuyiwa Prosper <notifications@github.com

wrote:

Shed more light on that

— Reply to this email directly or view it on GitHub https://github.com/andela/pibbble/issues/65#issuecomment-149701290.

unicodeveloper commented 8 years ago

Wow, then of course it wouldnt send because heroku has no idea about those constants, it has to be registered on the heroku dashboard as environment variables

ghost commented 8 years ago

So we push .env file then. Okay.

Oladipupo Isola | Software Developer, Andela See what TechCrunch http://techcrunch.com/2015/06/25/backed-by-spark-capital-andela-will-develop-a-continent-of-tech-talent/ and StackOverflow http://blog.stackoverflow.com/2015/05/stack-overflow-and-andela-partner-to-provide-education-beyond-borders/ say about us! (c) +234-703-302-0216 | Andela.com http://andela.com/

On Tue, Oct 20, 2015 at 10:21 PM, Otemuyiwa Prosper < notifications@github.com> wrote:

Wow, then of course it wouldnt send because heroku has no idea about those constants, it has to be registered on the heroku dashboard as environment variables

— Reply to this email directly or view it on GitHub https://github.com/andela/pibbble/issues/65#issuecomment-149707112.

unicodeveloper commented 8 years ago

No..you don't push the .env file, all we need to do is just register those constants on the server via heroku dashboard.

unicodeveloper commented 8 years ago

Slack me the constants and their values

ghost commented 8 years ago

Okay. I am on it.

Oladipupo Isola | Software Developer, Andela See what TechCrunch http://techcrunch.com/2015/06/25/backed-by-spark-capital-andela-will-develop-a-continent-of-tech-talent/ and StackOverflow http://blog.stackoverflow.com/2015/05/stack-overflow-and-andela-partner-to-provide-education-beyond-borders/ say about us! (c) +234-703-302-0216 | Andela.com http://andela.com/

On Tue, Oct 20, 2015 at 10:25 PM, Otemuyiwa Prosper < notifications@github.com> wrote:

Slack me the constants and their values

— Reply to this email directly or view it on GitHub https://github.com/andela/pibbble/issues/65#issuecomment-149708093.