Added body to events removed introduction/conclusion and properly display events on the show page #138
Conversation
|
|
||
| expect(rendered).to have_link "#{event.title}", href: event_path(event) | ||
| end | ||
| end |
| factory :event do | ||
| title "event title" | ||
| body "event content" | ||
| location |
|
rebased and ready to merge |
|
Hey @itsWill -- thanks for putting this together. I wanted to get this merged in tonight but there's an error when running the migration from the start: I know what the issue is, but a little tired now to think through a solution. Perhaps yourself or another contributor could take a look. Thanks! |
| @@ -1,4 +1,8 @@ | |||
| class CreateEvents < ActiveRecord::Migration | |||
| class Event < ActiveRecord::Base | |||
| translates :title, :introduction, :conclusion | |||
| end | |||
There was a problem hiding this comment.
The globalize translations are introduced in this migration then later removed, this causes an error in the rake db:migrate . This is a workaround to the issue @nicholasjhenry reported.
|
I added a work around to it, I managed to get it to work on my machine however without it, by doing a |
|
rebased, and ready to merge. |
|
@itsWill can you rebase again? :) |
| assign(:events, [event]) | ||
| render | ||
|
|
||
| expect(rendered).to have_link "#{event.title}", href: event_path(event) |
There was a problem hiding this comment.
Prefer to_s over string interpolation.
… events on index page
|
new -> #196 (that way I could commit on the branch) |
This fixes #106 and the outstanding tasks in #14
@nicholasjhenry