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

[DOC RELEASE] Ember 1.13 Component Lifecycle Events API Docs #13250

Merged
merged 1 commit into from
Apr 8, 2016

Conversation

Gaurav0
Copy link
Contributor

@Gaurav0 Gaurav0 commented Apr 4, 2016

Fixes #13244

screen shot 2016-04-04 at 5 17 38 pm

@Gaurav0 Gaurav0 changed the title Ember 1.13 Component Lifecycle Events API Docs [DOC RELEASE] Ember 1.13 Component Lifecycle Events API Docs Apr 4, 2016
@rwjblue
Copy link
Member

rwjblue commented Apr 4, 2016

This documents the evented versions of these, can we also document the methods themselves (which I think should be the primary way to use these hooks)?

@Gaurav0
Copy link
Contributor Author

Gaurav0 commented Apr 4, 2016

@rwjblue Sure. Done!

@rwjblue
Copy link
Member

rwjblue commented Apr 5, 2016

Looks like some JSCS errors with trailing white space are failing the build here. Can you take a look?

@Gaurav0
Copy link
Contributor Author

Gaurav0 commented Apr 5, 2016

@rwjblue Fixed.

@rwjblue
Copy link
Member

rwjblue commented Apr 6, 2016

@Gaurav0 - Would you mind squashing, and prefixing the remaining commit with [DOC release]?

This LGTM, @acorncom / @locks would one of y'all mind reviewing also?

@locks
Copy link
Contributor

locks commented Apr 6, 2016

I'm a bit iffy of so many view references, but I've read it and it looks good.

@rwjblue
Copy link
Member

rwjblue commented Apr 8, 2016

@locks - The screenshot says "view", but the diff uses "component" (the screenshot wasn't updated after the last round of changes).

@rwjblue rwjblue merged commit 47af7a2 into emberjs:master Apr 8, 2016
@Gaurav0 Gaurav0 deleted the component_event_docs branch April 8, 2016 01:17
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.

None yet

3 participants