Closed ColinBuyck closed 11 months ago
Name | Link |
---|---|
Latest commit | 3c89608408741471816d5785ff62f260765282ff |
Latest deploy log | https://app.netlify.com/sites/detroit-public-dev/deploys/658141e33128e70008015f53 |
Deploy Preview | https://deploy-preview-1648--detroit-public-dev.netlify.app |
Preview on mobile | Toggle QR Code...Use your smartphone camera to open QR code link. |
To edit notification comments on pull requests, go to your Netlify site configuration.
Name | Link |
---|---|
Latest commit | 3c89608408741471816d5785ff62f260765282ff |
Latest deploy log | https://app.netlify.com/sites/detroit-partners-dev/deploys/658141e3197689000800f87c |
Deploy Preview | https://deploy-preview-1648--detroit-partners-dev.netlify.app |
Preview on mobile | Toggle QR Code...Use your smartphone camera to open QR code link. |
To edit notification comments on pull requests, go to your Netlify site configuration.
Pull Request Template
Issue Overview
This PR addresses #1640 (BUT JUST NOW WITH DEV)
Description
This error is coming from attempting to sign the user in after they've successfully reset their password. Resolving this issue is still in progress but this PR will force users to sign-in again and gain access to the partners portal. Given the severity of the issue and scale of users locked out, I think this change is appropriate to buy more time for a comprehensive fix.
Note: I included messaging to the sign-in page about successfully updating password but that alert box will not show success site alert messages (without a change to UIC) so I chose a notice style as opposed to having no communication.
How Can This Be Tested/Reviewed?
This can be tested by creating an account with an email you have access to, following the forgot password flow until you are successfully logged in.
Checklist:
yarn generate:client
and/or created a migration if I made backend changes that require themReviewer Notes:
Steps to review a PR:
On Merge:
If you have one commit and message, squash. If you need each message to be applied, rebase and merge.