-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Implement pluck
operator
#1134
Comments
@trxcllnt is this going to be consistent with RxJS v4 with a recursive pluck? |
@mattpodwysocki sorry for the empty initial description, was in the middle of something but wanted to capture the issue anyway. I see no reason not to be backwards-compatible here. |
@trxcllnt are you working on this? I can take it if not. |
@luisgabriel ... I'm going to guess he is if he filed the issue. That's usually his thing. |
@Blesh Sure, I was suspecting that. It's worth asking anyway. ;) |
@luisgabriel I have not. In the middle of another project at work, feel free to take the lead on this. |
@mattpodwysocki I'm currently working on this, but I'm not sure if I understand what do you mean by recursive pluck correctly. Could elaborate a little more? |
@luisgabriel meaning that it is compatible with RxJS v4 |
Pretty similar to RxJS 4 implementation. Closes ReactiveX#1134
This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
edit: Implement
Observable.prototype.pluck
from RxJS v4.The text was updated successfully, but these errors were encountered: