-
Notifications
You must be signed in to change notification settings - Fork 49
Comments
Add pagination for wishlist_items. #141#151
Add pagination for wishlist_items. #141 #151nenadmil wants to merge 2 commits intorubyforgood:master from
Conversation
@leesharma
leesharma
left a comment
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.
Thank you for the PR! 🎉
Note: I set this up as a review app with 50 corgis: review app
The code looks good, but I've got a few bugs/change requests before merging:
-
Bug - On the wishlist page, it looks like all 50 items are shown (not just the first 10), despite the pagination links being at the bottom.
-
Request - The pagination links appear on the page, but they're not styled. Could you add the appropriate Bootstrap styles?
Current view: unstyled pagination
-
Request - This is a small one! Since the partials are so big, could you please change the default page size to 5?
Again, thank you for the PR! Let me know if you have any questions about my review comments!
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.
The config looks good!
Since our wishlist item partials are so large, could you please set the default to 5?
nenadmil
commented
Oct 23, 2017
Hi,
I fixed all, but there is a known problem with bullet that pops-up:
flyerhzm/bullet#147
Regards,
Nenad
Resolves #141
Add pagination for wishlist_items using kaminari gem.