-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
Replaced paginate for pagy in comments #8599
Conversation
.order('timestamp DESC') | ||
.where('node.status = ?', 1) | ||
.paginate(page: params[:page], per_page: 30) | ||
.where('node.status = ?', 1), items:30) |
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.
Space missing after colon.
Codecov Report
@@ Coverage Diff @@
## main #8599 +/- ##
==========================================
+ Coverage 81.76% 81.88% +0.11%
==========================================
Files 101 101
Lines 5901 5901
==========================================
+ Hits 4825 4832 +7
+ Misses 1076 1069 -7
|
Code Climate has analyzed commit f39ba13 and detected 0 issues on this pull request. View more on Code Climate. |
Hello @Tlazypanda, I replaced paginate for pagy in comments as well, hope that's okay. Kindly review it if it's okay |
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.
@stacytonui LGTM! 馃帀
Thanks you both 馃帀 |
* Replaced paginate for pagy * Replaced paginate for pagy
* Replaced paginate for pagy * Replaced paginate for pagy
* Replaced paginate for pagy * Replaced paginate for pagy
* Replaced paginate for pagy * Replaced paginate for pagy
* Replaced paginate for pagy * Replaced paginate for pagy
* Replaced paginate for pagy * Replaced paginate for pagy
Fixes #8426 (<=== Add issue number here)
Make sure these boxes are checked before your pull request (PR) is ready to be reviewed and merged. Thanks!
rake test
@publiclab/reviewers
for help, in a comment belowIf tests do fail, click on the red
X
to learn why by reading the logs.Please be sure you've reviewed our contribution guidelines at https://publiclab.org/contributing-to-public-lab-software
Thanks!