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

Updated README.md with newer syntax #179

Merged
merged 2 commits into from
Jul 29, 2015
Merged

Conversation

thec0keman
Copy link

This will have some overlap with the Ember guides, but since those are in process I wanted to take a stab at providing some documentation for how the unit / integration tests work in components.

This will have some overlap with the Ember guides, but since those are in process I wanted to take a stab at providing some documentation for how the unit / integration tests work in components.
@rwjblue
Copy link
Member

rwjblue commented Jul 24, 2015

Thanks for the PR! I'll try to review in detail soon, and give some feedback...

assert.ok(panel1.$().is(':visible'));

// render the component
this.render('{{ x-foo value=value action="result" }}');
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This misses the hbs inline template compiler function.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch!

@rwjblue
Copy link
Member

rwjblue commented Jul 29, 2015

This looks great!! I left some inline comments, but nothing major.

@thec0keman
Copy link
Author

@rwjblue / @martndemus thanks for the feedback. Changes should be good to go.

rwjblue added a commit that referenced this pull request Jul 29, 2015
Updated README.md with newer syntax
@rwjblue rwjblue merged commit b67c477 into emberjs:master Jul 29, 2015
@rwjblue
Copy link
Member

rwjblue commented Jul 29, 2015

Thank you!

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