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

Remove observer #70

Closed
wants to merge 1 commit into from
Closed

Remove observer #70

wants to merge 1 commit into from

Conversation

ondrejsevcik
Copy link

It's not necessary as the value is updated in didUpdateAttrs hook.

Should fix #67

@scottkidder
Copy link
Contributor

@ondrejsevcik If you rebase this should pass CI 👍

It's not necessary as the value is
updated in didUpdateAttrs hook.

Should fix #67
@ondrejsevcik
Copy link
Author

Rebased

@ondrejsevcik
Copy link
Author

Hi, any estimate when this will be merged? 🙏 thank you

@kennethkalmer
Copy link
Owner

kennethkalmer commented Oct 26, 2018 via email

@spansh
Copy link

spansh commented Jan 18, 2019

Any update on this, it's causing me some issues and I'd rather not have to fork.

@AndreJoaquim
Copy link

Hello @kennethkalmer! Any update on this? I think this is quite straightforward :)

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.

Observer is causing performance issues
5 participants