search

CoderDojoGitHub / our-dojo

Our dojo's website, used for archiving lesson plans and signing up students for classes
http://coderdojosf.com
5 stars 4 forks source link

Display multiple upcoming events #57

Closed mikefowler closed 10 years ago

mikefowler commented 10 years ago

Adds the ability to display multiple upcoming events on the homepage. In our case, and as implemented by these changes, we will display 2 upcoming events. The static Event.upcoming method now accepts a parameter, count, to support returning multiple upcoming events. count defaults to 1. Adds or updates appropriate tests, and slightly changes view logic in the home/index.html.erb view.

mikefowler commented 10 years ago

Attn @jonmagic, if you're around today.

jonmagic commented 10 years ago

A few small notes, looking great! Also :heart: the newsletter that went out today, looked great and the content was great.

mikefowler commented 10 years ago

Made suggested edits, @jonmagic. Press the magic button for me when verified.

mikefowler commented 10 years ago

Additional note: it seems like tests related to newsletter are failing, at least on my environment. Something off with my ENV setup or what?