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

Changed illegal access error to a warning #1529

Merged
merged 2 commits into from
May 22, 2018

Conversation

AmazingTurtle
Copy link
Contributor

The error described here: #1506 (Encountered an uncaught exception that was thrown by a reaction or observer component, in: 'Reaction[Autorun@163] Error: [mobx] Property 'isIntersection' of '[object Object]' was accessed through the prototype chain. Use 'decorate' instead to declare the prop or access it statically through it's owner) is now a warning instead.

On this PR someone should update the changelog, update docs and verify performance. Also the warning comes up every time, there is no flag that checks if the warning was already printed.

PR checklist:

  • Changed unit tests
  • Updated changelog
  • Updated docs (either in the description of this PR as markdown, or as separate PR on the gh-pages branch. Please refer to this PR). For new functionality, at least API.md should be updated
  • Added typescript typings
  • Verified that there is no significant performance drop (npm run perf)

Feel free to ask help with any of these boxes!

The above process doesn't apply to doc updates etc.

Changed observable object illegal access error to a warning instead
@AmazingTurtle AmazingTurtle changed the title Patch 1 Changed illegal access error to a warning May 4, 2018
@mweststrate mweststrate merged commit ac79559 into mobxjs:master May 22, 2018
@AmazingTurtle AmazingTurtle deleted the patch-1 branch May 22, 2018 11:10
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