Resolves issue #2544
Don't clear confirmation token, redirect to sign-in on repeated confi…
@markturner, this looks great, thank you! Could you please take a look at the build? This pull request is breaking it.
Fixing bug (oops)
Oops, sorry! I've fixed that now, except for just 2 remaining tests where I'm not quite sure what's happening.
I'll take another look over the weekend unless you or someone else can figure it out.
Sorry, I wasn't able to figure it out and I'm not going to be able to put anymore time into it. It's probably obvious to someone else but my testing knowledge is pretty much all rspec! Would be great if someone with more of a handle on this than me could get these last two tests to pass.
Thanks @markturner, don't worry. We will get around to it and merge your fixes in.
Looks like this has already been integrated?
No, I don't think it has yet.
Is this still relevant?
@markturner Cold you please rebase your branch? Thanks.