mobx
mobx copied to clipboard
fix: Allow action field decorators without makeObservable (#3879)
Code change checklist
- [x] Added/updated unit tests
- [x] Updated
/docs. For new functionality, at leastAPI.mdshould be updated - [x] Verified that there is no significant performance drop (
yarn mobx test:performance)
Similar to #3883 (credit to @Obi-Dann ) but add extra handling for @action.bound on field, for example
class A {
@action.bound
doSomething = function() {...} // `this` should be bound to instance of class A
}
fix #3879
🦋 Changeset detected
Latest commit: b6b84df0943db1bc6ba9a3345dfacd5075b3d411
The changes in this PR will be included in the next version bump.
This PR includes changesets to release 1 package
| Name | Type |
|---|---|
| mobx | Patch |
Not sure what this means? Click here to learn what changesets are.
Click here if you're a maintainer who wants to add another changeset to this PR
Hi @Matchlighter I updated doc to explain the difference behaviour between legacy and modern also added a new unit test to demonstrate the difference