LukeBrandon / fluke

5 stars 0 forks source link

Db user setup #26

Closed FlynnOConnell closed 1 year ago

FlynnOConnell commented 1 year ago

Main user registration validation logic:

LukeBrandon commented 1 year ago

Resolved all requested changes.

I did ignore one suggestion about:

Ok(Created::new(location) because this is exactly what I did, just not in the commented function. I think leaving the logic to attempt the "create_user" query, and the logic to handle the resulting responses from that query are better left separate. signup_user handles that logic in the current version.

I didn't realize it was being called by something else, that is good then