search

aldrichdev / neatf2p-nextjs

Next.js project/website for Neat F2P.
https://neatf2p-nextjs.vercel.app
0 stars 1 forks source link

F2P-139 | Potential issue with `updateWebsiteUserEmailAddress` #121

Closed aldrichdev closed 2 weeks ago

aldrichdev commented 2 months ago

What's Changed

Testing Notes

Confirmed an email is sent to the old email address when a website user's email is changed, and the template variable works (and shows the new address).

vercel[bot] commented 2 months ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
neatf2p-nextjs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 28, 2024 8:20pm
aldrichdev commented 2 months ago

I think this ticket needs more iterations. I have noticed a few issues.

  1. updateWebsiteUserEmailAddress runs multiple times in change-email/[id]. I have fixed this in a local stash.
  2. I added code to log the user out and then back in when the notice email is sent to the old address; however, this doesn't work consistently. Sometimes we get 403 Forbidden on the next email update. Other times I get Access Denied when navigating to the account page after the update is successful.
  3. The notice email to the old address is never sent to charadicks@gmail.com every time I've changed my email from that to another one. It looks like they were sent to null@gmail.com since the notices show up in the Neat F2P gmail account: image