mobxjs / mobx-state-tree

Full-featured reactive state management without the boilerplate
https://mobx-state-tree.js.org/
MIT License
6.9k stars 639 forks source link

Add tests and documentation for new parent argument to `types.optional` #2056

Closed coolsoftwaretyler closed 10 months ago

coolsoftwaretyler commented 11 months ago

In https://github.com/mobxjs/mobx-state-tree/pull/2050, we added an new way for types.optional to access their parent context in the function argument.

We should add tests to cover these new cases. The PR author wrote up some possible examples along with context there.

coolsoftwaretyler commented 10 months ago

We are going to actually revert #2050, as per this comment thread - that makes this issue irrelevant. Closing it out.