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

Consecutive next listeners do not gets fired #1

Open
ranajitbanerjee opened this issue Nov 5, 2018 · 1 comment
Open

Consecutive next listeners do not gets fired #1

ranajitbanerjee opened this issue Nov 5, 2018 · 1 comment
Assignees
@adotg

Comments

@ranajitbanerjee
Copy link
Member

ranajitbanerjee commented Nov 5, 2018 •
edited
Loading

When registering a next listener of trigger of another next listener, the new next listener function does not get called when the property is changed in the store.

To reproduce the issue, check this link, open the console and you can see only range.start message gets displayed, range.end message does not gets displayed.
https://jsfiddle.net/ykveqzra/4/
@adotg adotg self-assigned this Nov 5, 2018
@adotg adotg changed the title Next listener does not trigger Consecutive next listeners do not gets fired Nov 5, 2018
@adotg
Copy link
Contributor

adotg commented Nov 12, 2018

Branch feature/#1-fix-consecurive-next-listeners is updated with code. However, for listeners which get fired from inside another next listener, receive incorrect old value.

adotg added a commit that referenced this issue Jan 30, 2019
@adotg
Merge pull request #2 from chartshq/feature/#1-fix-consecurive-next-l…
6c7d636 
…isteners

Feature/#1 fix consecurive next listeners
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@adotg adotg

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@adotg @ranajitbanerjee