Skip to content
This repository was archived by the owner on Dec 4, 2017. It is now read-only.

docs(server-communication) Don't use constructor, but rather ngOnInit #3046

Merged
merged 2 commits into from
Jan 1, 2017
Merged

docs(server-communication) Don't use constructor, but rather ngOnInit #3046

merged 2 commits into from
Jan 1, 2017

Conversation

wKoza
Copy link
Contributor

@wKoza wKoza commented Dec 29, 2016

Issue #3045

Foxandxss
Foxandxss approved these changes Dec 29, 2016
View reviewed changes
Copy link
Member

@Foxandxss Foxandxss left a comment

Choose a reason for hiding this comment

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

I do like this change.

@wardbell
Copy link
Contributor

wardbell commented Jan 1, 2017

I do too. Merging.

@wardbell wardbell merged commit 7c9ff12 into angular:master Jan 1, 2017
abdel-ships-it pushed a commit to abdel-ships-it/angular.io that referenced this pull request Feb 11, 2017
@wKoza @abdel-ships-it
docs(server-communication) Don't use constructor, but rather ngOnInit (…
c6c3e68 
…angular#3046)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@Foxandxss Foxandxss Foxandxss approved these changes

Assignees
No one assigned
Labels
CLA: yes
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@wKoza @wardbell @Foxandxss @googlebot