search

dcramer / peated

https://peated.com
Apache License 2.0
63 stars 13 forks source link

Bug: Changing username leads to 404 #102

Closed JacobCoffee closed 10 months ago

JacobCoffee commented 10 months ago

Reproduce

  1. New account
  2. Change username
  3. Click save
  4. Go to the top right profile link
  5. Click, 404
  6. See that the href is the only one image
  7. Manually browse to https://peated.com/users/$new_username
  8. $$$
dcramer commented 10 months ago

Need to repro in production, but maybe this coincidentally fixed itself with the tRPC conversion. I dont think the code paths should have changed to re-bind the current session data, but it at least appears to work in dev for me. I tried real quick on prod and it seemed to work too. Possible there's more to it than this, but at least for now we'll assume this was fixed w/ the massive API rewrite.