Upgrade to Mobx 6 from version 5

[besLisbeth]

Hi there 👋,
I've got some problems with the upgrade to version 6. All my @action.bound shows that this is undefined. I use them in the classes in the next construction:

export default class Store {
    @observable value1 = [];
    @observable value2 = [];

    @action.bound
    myFn() {
        this.anotherFnCall(); // <- this is undefined here
    }
}

Does anyone also have any issues with them?

[danielkcz]

Please follow the issue template which also mentions the migration guide :/

[besLisbeth]

@FredyC, I'm really sorry, but I didn't find anything which could cause such an issue in the new migration guide or CHANGELOG. Was that caused by strict mode?

[mweststrate]

I'm not sure how you can have read all those in < 7 minutes...

https://mobx.js.org/migrating-from-4-or-5.html#upgrading-classes-to-use-makeobservable

[besLisbeth]

Thank you a lot, I really blame myself for not reading it up attentively.
But, maybe, it will be helpful to also add warning to the console, as the errors I got didn't look like the problem with decorators?
Thanks, anyway.

[mweststrate]

we would if we could. But it is hard to present things not being present :)

…

On Tue, Oct 20, 2020 at 1:11 PM Yelyzaveta Boiko ***@***.***> wrote: Thank you a lot, I really blame myself for not reading it up attentively. But, maybe, it will be helpful to also add warning to the console, as the errors I got didn't look like the problem with decorators? Thanks, anyway. — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#2538 (reply in thread)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAN4NBATIYGTXTTQ65Q76PTSLV46TANCNFSM4SX2QIBA> .