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

vue-observable status #33

Closed
zoomclub opened this issue Mar 16, 2017 · 5 comments
Closed

vue-observable status #33

zoomclub opened this issue Mar 16, 2017 · 5 comments

Comments

@zoomclub
Copy link

zoomclub commented Mar 16, 2017

@yyx990803 @regou I'm liking directives more and more and so I'm keen on seeing the v-ob directive proposed in issue #14 become part of vue-rx/observable. Can it be merged in or is there some problem with it?

In reviewing various vue-rx issues there seem to be a few more suggestions raised. Maybe these be resolved into a next release? We are almost at 500 stars so that should mean that this room is getting brighter for sure.

After preparing other aspects of my project I'm back to taking a look at how the Vue reactivity engine and the Observable engine can work in a harmonious way, my specific requirements are always the catalyst. Have some new strategies I'm working through in my project, always interested in discussion.

@yyx990803
Copy link
Member

This is just not a high priority item for me at the moment. Open to community efforts though, obviously.

@regou
Copy link
Collaborator

regou commented Mar 16, 2017

@zoomclub See the 3rd item on this issue comment so no merge for that.
I been think a lot about this directive, how about this new one:

example

v-stream directive code

if the usage is ok, I'll create a pull request

@zoomclub
Copy link
Author

zoomclub commented Mar 16, 2017

@yyx990803 Thanks for the notice, will keep on keeping on then 💯

@zoomclub
Copy link
Author

@regou The new v-stream directive looks great! Good name too, seeing that other stream libraries can also be supported. Look forward to your PR 👍

@zoomclub
Copy link
Author

To celebrate the resolution of this issue and the release of vue-rx 3.0 I'm closing this issue 🍾

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

No branches or pull requests

3 participants