View example did not work #232

Closed
wants to merge 1 commit into
from

Projects

None yet

2 participants

@tow8ie
tow8ie commented Dec 14, 2012

The underscore.js template function only got passed the jQuery object, not the template string.

@addyosmani
Owner

Is there a reason you opted for .text() over .html()?

No, just lack of experience which is to prefer.

Owner

No worries! We'll get this fixed up. Appreciate the heads up.

@addyosmani
Owner

Thanks! I've left a comment inline.

@addyosmani addyosmani added a commit that closed this pull request Dec 14, 2012
@addyosmani Fixes #232 f5db307
@addyosmani addyosmani added a commit that referenced this pull request Dec 23, 2012
@addyosmani Merge commit 'f5db307a70df6a64cd6c94aa64e77f5d418400ce' into gh-pages
* commit 'f5db307a70df6a64cd6c94aa64e77f5d418400ce':
  Fixes #232
991aa8d
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment