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

Removing event.current #100

Closed
rpominov opened this issue May 5, 2015 · 0 comments
Closed

Removing event.current #100

rpominov opened this issue May 5, 2015 · 0 comments
Labels

Comments

@rpominov
Copy link
Member

rpominov commented May 5, 2015

We have Events in Kefir. They are objects with interface described in .onAny documentation and also used in the .withHandler method.

Each event has boolean .current property. This property going to be removed in v3.0.0. Turned out we don't need it internally, and I doubt anyone depend on it in user land.

Migration

Before

obs.onAny(event => {
  console.log(event.current ? 'current' : 'normal', event.value);
});

After

var current = true;
obs.onAny(event => {
  console.log(current ? 'current' : 'normal', event.value);
});
current = false;
@rpominov rpominov added this to the v3.0.0 milestone May 5, 2015
rpominov added a commit that referenced this issue May 6, 2015
@rpominov rpominov mentioned this issue Sep 3, 2015
9 tasks
rpominov added a commit that referenced this issue Sep 5, 2015
rpominov added a commit that referenced this issue Sep 5, 2015
rpominov added a commit that referenced this issue Sep 6, 2015
rpominov added a commit that referenced this issue Sep 19, 2015
@rpominov rpominov removed this from the v3.0.0 milestone Sep 19, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant