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

[edge guides] 5.2 Read order #21121

Closed
PierreVannier opened this issue Aug 4, 2015 · 3 comments
Closed

[edge guides] 5.2 Read order #21121

PierreVannier opened this issue Aug 4, 2015 · 3 comments

Comments

@PierreVannier
Copy link

4th code exerpt is :

# find all users named David who are Code Artists and sort by created_at in reverse chronological order
users = User.where(name: 'David', occupation: 'Code Artist').order('created_at DESC')

whereas it probably could be :

# find all users named David who are Code Artists and sort by created_at in reverse chronological order
users = User.where(name: 'David', occupation: 'Code Artist').order(created_at: :desc)
@kaspth
Copy link
Contributor

kaspth commented Aug 4, 2015

Nice catch! Do you want to send us a pull request? 😄

@senny
Copy link
Member

senny commented Aug 10, 2015

@PierreVannier updated. Thank you for reporting 💛

@senny senny closed this as completed in 8d70f40 Aug 10, 2015
@PierreVannier
Copy link
Author

👍🏻😉

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

No branches or pull requests

4 participants