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.

Sign up for GitHub

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

Jump to bottom

Paginate NewsEvent on homepage #102

Open
benichu opened this issue Dec 4, 2015 · 8 comments
Open

Paginate NewsEvent on homepage #102

benichu opened this issue Dec 4, 2015 · 8 comments
Assignees
@cawel
Labels
feature

Comments

@benichu
Copy link
Member

benichu commented Dec 4, 2015

  • display 10 most recent
  • add pagination menu
@cawel
Copy link

cawel commented Dec 4, 2015

Using kaminari or will_paginate?

@benichu
Copy link
Member Author

benichu commented Dec 4, 2015

Your choice... Choose carefully ;)

@mathieugagne
Copy link
Member

My vote goest to kaminari

@cawel
Copy link

cawel commented Dec 4, 2015

Same here!

@cawel cawel self-assigned this Dec 4, 2015
@nicholasjhenry
Copy link
Contributor

@benichu Does #48 cover this requirement?

@cawel
Copy link

cawel commented Dec 4, 2015

My take on it: I guess they should be 2 separate issues, but #48 should be done before #102.

@benichu
Copy link
Member Author

benichu commented Dec 4, 2015

👍 @cawel

@benichu benichu modified the milestone: Site live for Meetup Dec 4, 2015
@benichu benichu removed this from the Site live for Meetup milestone Dec 12, 2015
@steakunderscore
Copy link
Member

I have put a patch up for this, but it doesn't include merging the events and news. So for now I won't make a PR. steakunderscore@6b831e3

@sophiedeziel sophiedeziel removed the site label Feb 26, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@cawel cawel

Labels
feature
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
6 participants
@benichu @cawel @nicholasjhenry @steakunderscore @mathieugagne @sophiedeziel