-
Notifications
You must be signed in to change notification settings - Fork 656
Port categories/slugs over to Diesel #759
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
359d405
to
3736001
Compare
let rows = stmt.query(&[])?; | ||
let conn = req.db_conn()?; | ||
let slugs = categories::table | ||
.select((categories::slug, categories::slug)) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd still like the order by to ensure that the slugs are alphabetical, please :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lol wtf why did I write this
3736001
to
b87d7b2
Compare
Updated and added a test for this endpoint. |
b87d7b2
to
c15c897
Compare
c15c897
to
e664df5
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I rebased since I changed some things out from under you :) This looks great now, just waiting for travis to go green then I'll merge!
Failure was just |
No description provided.