X Tutup
Skip to content

feat(query): implement ContentChildren and ViewChildren#4237

Closed
vsavkin wants to merge 2 commits intoangular:masterfrom
vsavkin:query_to_field
Closed

feat(query): implement ContentChildren and ViewChildren#4237
vsavkin wants to merge 2 commits intoangular:masterfrom
vsavkin:query_to_field

Conversation

@vsavkin
Copy link
Copy Markdown
Contributor

@vsavkin vsavkin commented Sep 17, 2015

@mhever @rkirov Could either of you review this guy?

@vsavkin vsavkin added action: review The PR is still awaiting reviews from at least one requested reviewer effort2: days labels Sep 17, 2015
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

did we consider having queries and viewqueries separately?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Nevermind, there is no value is separating them.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Replace Function event with (any, any) => any it will still provide a bit more typechecking, i.e. calling it with wrong number of args.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Fixed it.

@rkirov rkirov added pr_state: LGTM and removed action: review The PR is still awaiting reviews from at least one requested reviewer labels Sep 17, 2015
@vsavkin vsavkin added the action: merge The PR is ready for merge by the caretaker label Sep 18, 2015
@mary-poppins
Copy link
Copy Markdown

Merging PR #4237 on behalf of @vsavkin to branch presubmit-vsavkin-pr-4237.

@mary-poppins mary-poppins removed the action: merge The PR is ready for merge by the caretaker label Sep 18, 2015
@rkirov rkirov mentioned this pull request Sep 18, 2015
12 tasks
@vsavkin vsavkin added the action: merge The PR is ready for merge by the caretaker label Sep 20, 2015
@mary-poppins
Copy link
Copy Markdown

Merging PR #4237 on behalf of @vsavkin to branch presubmit-vsavkin-pr-4237.

@mary-poppins mary-poppins removed the action: merge The PR is ready for merge by the caretaker label Sep 20, 2015
@vsavkin vsavkin closed this in 2e9de0b Sep 20, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants

X Tutup