Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Solve issue when stealing mustache templates #175

Merged
merged 1 commit into from
Nov 27, 2012
Merged

Solve issue when stealing mustache templates #175

merged 1 commit into from
Nov 27, 2012

Conversation

marioquirino
Copy link

Solve issue when stealing mustache templates.
Source: https://forum.javascriptmvc.com/topic/mustache-rendering-error

@daffl
Copy link
Contributor

daffl commented Nov 27, 2012

Thanks. This will also close #164. I'm wondering why this is necessary.

@marioquirino
Copy link
Author

I checked and it seems that mustache.js was the only one that was returning a different object.
ejs.js, jaml.js, micro.js and tmpl.js were all returning a can object.

Whatever is receiving the template engine (I can't recall where I saw it) to compile the mustache templates, it expects an object as argument, and that object has a property called "Mustache", which is, in this case, the can object.

Regarding the thing about modules, I'm not sure how I can help with that though. I'm still trying to get how modules and AMD stuff works.

daffl added a commit that referenced this pull request Nov 27, 2012
Solve issue when stealing mustache templates
@daffl daffl merged commit 64e7e06 into canjs:master Nov 27, 2012
@daffl
Copy link
Contributor

daffl commented Nov 27, 2012

Seems to work fine. Anybody who is using Steal + 1.1.1 need to change their Mustache object reference but it solves the problem. Thanks again!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants