Skip to content

Resolves #6 - removed updateComponentState for all events. #7

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Biohazord
Copy link
Contributor

This is the change we're currently using in product to deal with an issue where a view is destroyed and then recreated.

@Biohazord
Copy link
Contributor Author

Hopefully I created this PR properly.

@Biohazord
Copy link
Contributor Author

It looks like Travis CI had issues with running the npm install command. Can you please verify/validate that the subsequent run 41.2 completed properly?

@imktks
Copy link
Contributor

imktks commented Nov 13, 2019

@Biohazord you are absolutely right about the issue with listeners. I will check it asap and i will get back to you. Thanks!

@imktks
Copy link
Contributor

imktks commented Nov 13, 2019

Travis should be ok now can you please rebase your branch with master ?

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

Successfully merging this pull request may close these issues.

2 participants