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

[BUGFIX beta] Added assert mismatched template compiler version. #3729

Merged
merged 1 commit into from
Nov 20, 2013

Conversation

machty
Copy link
Contributor

@machty machty commented Nov 12, 2013

See #3728. Using an old version of Ember to precompile templates
can result in an unusual error when trying to render a component
in block form, e.g.

  {{#my-component}}Hello{{/my-component}}

This commit adds an assert to give the user more information
as to what's going on.

See emberjs#3728. Using an old version of Ember to precompile templates
can result in an unusual error when trying to render a component
in block form, e.g.

```
  {{#my-component}}Hello{{/my-component}}
```

This commit adds an assert to give the user more information
as to what's going on.
@machty
Copy link
Contributor Author

machty commented Nov 12, 2013

/cc @trek

@wagenet
Copy link
Member

wagenet commented Nov 12, 2013

For Handlebars, we have a COMPILER_REVISION that we check for mismatches. Maybe we need the same for Ember Handlebars too. It seems like this would be more reliable than attempting to catch all the cases where things go wrong.

@machty
Copy link
Contributor Author

machty commented Nov 15, 2013

I'm gonna do what @wagenet is suggesting

@rwjblue
Copy link
Member

rwjblue commented Nov 16, 2013

@machty - Do you think the fix you mention will make it into 1.2.0 (current beta)?

@machty
Copy link
Contributor Author

machty commented Nov 16, 2013

I can't see a good hook into Handlebars to perform this check and I don't think we should wait for a Handlebars PR to come through. Reopening, as I think this is probably the best solution so that people at least understand why they're getting trolled.

@machty machty reopened this Nov 16, 2013
@machty
Copy link
Contributor Author

machty commented Nov 16, 2013

@rjackson I think it should, waiting on feedback from some more homies.

machty added a commit that referenced this pull request Nov 20, 2013
[BUGFIX beta] Added assert mismatched template compiler version.
@machty machty merged commit ea261c9 into emberjs:master Nov 20, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants