[snewcomer]

What's in this PR?

password reset controller finding AuthToken.

@begedin should I also change the AuthToken model value to token?

References

Fixes #731

Progress on: #498

[snewcomer]

Wondering if we can just look up the token or should look up the token/user in the AuthToken model.

[joshsmith]

This is looking pretty good. Where did tests go down?

[begedin]

Looks good, @snewcomer

There is no need to change the property name in the model itself. My suggestion was just about having the public API interface be clear to the outside client. The model is internal to the API and should be clearer to the dev, so auth_token.value will make more sense that way. This is good to go.

[begedin]

@joshsmith Looks like tests going down is coveralls being coveralls. It reports them going down in files untouched by this PR.

@snewcomer You'll have to rebase and rerun migrations to deal with the structure.sql conflict, but this is good to merge after.

[snewcomer]

Going to squash and merge!

[joshsmith]

🎉 yay!