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

ClayAlert disposed instances issues #582

Closed
carloslancha opened this issue Feb 20, 2018 · 0 comments
Closed

ClayAlert disposed instances issues #582

carloslancha opened this issue Feb 20, 2018 · 0 comments
Assignees
@matuzalemsteles
Labels
status: next-release Issues that will enter into the next release

Comments

@carloslancha
Copy link
Contributor

We need to look into ClayAlert, ClayToast and ClayStripe to investigate the error

    console.warn node_modules/metal-state/lib/State.js:788
      Error. Trying to access property "_visible" on disposed instance
    console.warn node_modules/metal-state/lib/State.js:788
      Error. Trying to access property "destroyOnHide" on disposed instance
@matuzalemsteles matuzalemsteles self-assigned this Feb 22, 2018
carloslancha added a commit that referenced this issue Feb 22, 2018
@carloslancha
Merge pull request #625 from matuzalemsteles/issue-582
0d7333e 
Check if the dispose has already been done and does nothing | Fixes #582
@matuzalemsteles matuzalemsteles added the status: next-release Issues that will enter into the next release label Feb 22, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@matuzalemsteles matuzalemsteles

Labels
status: next-release Issues that will enter into the next release
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@carloslancha @matuzalemsteles