railsbridge/bridge_troll — pull request #42 by wholenotes (2012-05-01 04:16:13+00:00)
Title: Completed minor adjustments in response to your comments

Changed the following per our discussion:

event model:
if @rsvp and @rsvp.valid?
Throw the @attr directly into the where method ... unless you are using it in two places

events controller spec:
This will be more brittle as the front end text changes. Let's change to /thanks/i ... which is not part of the unvolunteer flash

event model, controller, view, specs
when changing data, use ! methods: #volunteer! and #unvolunteer!

event view:

Lets clean up the if stuff

if user_signed_in?
if event.volunteering?(current_user)
# ...
else
# ...
end
end