Infrastructure for Snowdrift.coop. This is a MIRROR of https://gitlab.com/snowdrift/snowdrift. Your issue reports and merge requests are welcome, but they will be moved to gitlab.com. You are encouraged to start there instead!
Hi, I'm just getting familiar with this codebase (got pointed here from CodeTriage) and hoping to tackle #144. I noticed when I first created a test account that inputting an invalid authentication token redirected me to /auth/login, upon which I had to use my browser "back" button to get back to /auth/verify-account and input the correct token. Obviously this is not a huge issue, but it's awkward from a UI perspective.
Steps to reproduce:
visit /auth/create-account
enter an email address and passphrase, and continue on to /auth/verify-account
enter gibberish in the "Token" box
get redirected to /auth/login upon which you are greeted by a yellow text box and a concerned dog
In my opinion, the ideal behavior here would be just to redirect to /auth/verify-account (and display the yellow box and concerned dog) until the user manages to enter the correct token. Thoughts?
Edit: Since this is such a small change, I'm going to close this issue and put in a pull request. Then you can just dump it if it's not what you have in mind.
Hi, I'm just getting familiar with this codebase (got pointed here from CodeTriage) and hoping to tackle #144. I noticed when I first created a test account that inputting an invalid authentication token redirected me to /auth/login, upon which I had to use my browser "back" button to get back to /auth/verify-account and input the correct token. Obviously this is not a huge issue, but it's awkward from a UI perspective.
Steps to reproduce:
/auth/create-account/auth/verify-account/auth/loginupon which you are greeted by a yellow text box and a concerned dogIn my opinion, the ideal behavior here would be just to redirect to
/auth/verify-account(and display the yellow box and concerned dog) until the user manages to enter the correct token. Thoughts?Edit: Since this is such a small change, I'm going to close this issue and put in a pull request. Then you can just dump it if it's not what you have in mind.