Commit graph

9 commits

Author SHA1 Message Date
816584f177 Move item_link partial caching to a helper rather than the template itself
This is a surprisingly huge performance gain. On my testing (with
cache_classes set to true to also cache templates), this sped up
closet_hangers#index rendering by a factor of 2 when there were a
significant number of items. Cool beans.

I think we can even hold off on the individual hanger caching now:
we've made the closet hanger partial tons faster by moving forms out
of them and doing this cache check earlier. I'm expecting significant
performance gains both here and on items#index (though less so there).
I'll deploy and see how much it helps in production; if not enough, we
can look at the layered caching of hangers, lists, groups, full pages,
etc.

So glad we don't *have* to move to a pagination model!
2013-06-22 16:31:46 -07:00
1e3938eea9 improve closet performance by caching item link 2012-08-09 19:34:56 -04:00
ea7171b322 fix ambiguous item_link partial throwing errors in outfits#show 2011-07-31 23:45:57 -04:00
d5641dddbb user closet display 2011-07-14 13:56:20 -04:00
3c473bee2c nice pretty upgrade to item show page 2010-09-08 22:49:39 -04:00
cc3a648e48 move closet to redesign 2010-06-30 14:51:49 -04:00
df3fcd101f retain query on item page 2010-06-07 20:02:46 -04:00
f794d07c2c basic show page 2010-05-15 20:46:41 -04:00
8519f9131b item presentation is neat 2010-05-15 14:01:15 -04:00