12spokes/tandem — pull request #33 by treybean (2012-03-01 02:04:46+00:00)
Title: [Ethan] Tandem helper bug

https://www.pivotaltracker.com/story/show/25743221
https://www.pivotaltracker.com/story/show/25735537

When integrating with Flash Card, we noticed cucumber scenarios were failing because tandem_navigation_tag wasn't available on /users/sign_in. Investigating further, I discovered that this only happened on the first request. Colleen and I observed it on the FC app yesterday, but since a refresh solved it, we thought maybe it wasn't a big deal.

I wasn't able to reproduce this in the tandem dummy app, though, which also uses devise. I tried making sure rails and devise versions were the same, but couldn't get it to occur on that app. Not sure why. I ended up testing the FC app against my local tandem to verify the fix worked.

Once I got the tandem helpers showing up on the initial request, another bug showed itself. If there aren't any tandem pages in the database, the pages#home action fails even though it was coded as if it would work. Now, we go ahead and create a page for you. I think we'll now be able to get rid of the persisted? checks in a few places in the app as well. Chore'd.