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

Don't directly fetch in database helpers #35

Open
barzamin opened this issue Apr 1, 2018 · 0 comments
Open

Don't directly fetch in database helpers #35

barzamin opened this issue Apr 1, 2018 · 0 comments
Assignees
Labels
A: Backend Anything related to the backend C: Enhancement ✨ An issue which details an improvement to the software, not necessarily to fix a bug. P: Medium ☕ Medium priority ticket

Comments

@barzamin
Copy link
Member

barzamin commented Apr 1, 2018

If we don't implicitly perform fetches in database helpers like User::by_id, we can compose them: see this chunk of the Diesel docs.

@barzamin barzamin added C: Enhancement ✨ An issue which details an improvement to the software, not necessarily to fix a bug. P: Medium ☕ Medium priority ticket labels Apr 1, 2018
@barzamin barzamin self-assigned this Jun 29, 2018
@barzamin barzamin added A: Backend Anything related to the backend and removed A-database labels Jul 23, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A: Backend Anything related to the backend C: Enhancement ✨ An issue which details an improvement to the software, not necessarily to fix a bug. P: Medium ☕ Medium priority ticket
Projects
None yet
Development

No branches or pull requests

1 participant