Skip to content
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

Use knexjs instead of raw queries #67

Closed
knownasilya opened this issue Aug 5, 2014 · 3 comments
Closed

Use knexjs instead of raw queries #67

knownasilya opened this issue Aug 5, 2014 · 3 comments
Labels

Comments

@knownasilya
Copy link

Seems like the queries are pretty standard, which would make them an ideal candidate for knexjs. Knex supports the same databases, which would cut your code base down so there is less to manage.

@simov
Copy link
Owner

simov commented Aug 5, 2014

Thanks @knownasilya I'll take a look at it!

@simov simov added the question label Aug 5, 2014
@simov
Copy link
Owner

simov commented Aug 31, 2014

I ended up rolling my own query builder instead. Also I moved all query generation related code to one common place https://github.com/simov/express-admin/tree/master/lib/qb

@simov simov closed this as completed Aug 31, 2014
@knownasilya
Copy link
Author

@simov nice!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants