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

Jump to bottom

Password Reset Causes 500 Error on Production #108

Closed
vkoves opened this issue Sep 27, 2018 · 4 comments
Closed

Password Reset Causes 500 Error on Production #108

vkoves opened this issue Sep 27, 2018 · 4 comments
Assignees
@tkimia
Labels
bug An issue how in how the project works good first issue Issues new contributors can easily work on help wanted Issues we want help on
Milestone
Minimum Viable Pr...

Comments

@vkoves
Copy link
Contributor

vkoves commented Sep 27, 2018

Detailed Description

Trying to reset my password fails and throws a 500 error. Looking at Papertrail, I see this:

screenshot from 2018-09-27 18-15-12

Considering I have fixed our SendGrid account and updated all of the credentials, this is likely an error in the prod configuration of SendGrid.

Context

Resetting your password is an MVP feature that was deployed in PR #96, and should be working.

Possible Implementation

See SendGrid addon docs.

Your Environment

Not relevant.
@vkoves vkoves added the bug An issue how in how the project works label Sep 27, 2018
@vkoves vkoves self-assigned this Sep 27, 2018
@vkoves vkoves added this to the Minimum Viable Product milestone Oct 10, 2018
@SorenSpicknall SorenSpicknall added help wanted Issues we want help on good first issue Issues new contributors can easily work on not started labels Feb 13, 2019
@vkoves
Copy link
Contributor Author

vkoves commented Feb 16, 2019

@AcademyInPeril - can you unassign me from this issue? I don't think I'll be taking it on in the near future

@vkoves
Copy link
Contributor Author

vkoves commented Feb 24, 2019

@AcademyInPeril - pinging you on this

@EvanGrillo
Copy link
Contributor

EvanGrillo commented Mar 13, 2019 •
edited

Password reset to work without error will require that we set up a mailer - would be happy to work on this. Need email/password

@tkimia tkimia assigned tkimia and unassigned vkoves Sep 18, 2019
vkoves added a commit that referenced this issue Sep 20, 2019
@vkoves
Merge pull request #173 from TransitTalk/bugfix/reset-password-500
07e8734 
 #108 - fix email sending in production
@vkoves vkoves mentioned this issue Sep 20, 2019
Merged
@vkoves
Copy link
Contributor Author

vkoves commented Sep 20, 2019

Fixed in #173, which I confirmed on prod, so I am closing this issue as fixed!

@vkoves vkoves closed this as completed Sep 20, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tkimia tkimia

Labels
bug An issue how in how the project works good first issue Issues new contributors can easily work on help wanted Issues we want help on
Projects
None yet
Milestone
Minimum Viable Product
Development

No branches or pull requests
4 participants
@tkimia @vkoves @SorenSpicknall @EvanGrillo