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.

Sign up for GitHub

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

Jump to bottom

Switch to Redux observable implementation #123

Closed
stubailo opened this issue Apr 18, 2016 · 4 comments
Closed

Switch to Redux observable implementation #123

stubailo opened this issue Apr 18, 2016 · 4 comments

Comments

@stubailo
Copy link
Contributor

reduxjs/redux#1632

Then maybe we won't need our own shim!
@stubailo stubailo added the idea label Apr 18, 2016
@stubailo stubailo mentioned this issue Apr 18, 2016
Merged
2 tasks
@stubailo stubailo changed the title Switch to Redux observable implementation when merged Switch to Redux observable implementation Apr 19, 2016
@arunoda
Copy link

arunoda commented Apr 22, 2016

You mean Rxjs?

@stubailo
Copy link
Contributor Author

No, redux has built in observables now. I feel like Rxjs observables are way too heavy to be a standard part of the client, but we should make it possible to use them!

@arunoda
Copy link

arunoda commented Apr 22, 2016

Okay. We'll work on a simple wrapper for Rx.

@stubailo
Copy link
Contributor Author

OK it turns out that using that won't help us at all, because the shim for observable is very simple. Closing in favor of #149 for RxJS.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 17, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@arunoda @stubailo