This repository has been archived by the owner on Apr 7, 2021. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 4
peer dependency mobx 4 ? is mobx 5 unsupported? #16
Comments
MobX 5 is not supported yet. Some decorators depend on |
I am not so sure since I am using |
`spy` is dev only feature in mobx 5 and has no effect in production build.
Decorators can be rewrite to work without using spy but I will have time
for this in several weeks only.
…On Sat, Aug 18, 2018, 10:51 AM Jigar Gosar ***@***.***> wrote:
but spy was removed in v5
I am not so sure since I am using spy in v5, let me know. Thanks.
https://mobx.js.org/refguide/spy.html
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#16 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ACPAvRR7jL30yHm8Wc8k3XmtdpF4OT9Gks5uR8dlgaJpZM4WBFx6>
.
|
thanks. there is no rush. I was just curious.
On Mon, 20 Aug 2018 at 15:56 farwayer ***@***.***> wrote:
`spy` is dev only feature in mobx 5 and has no effect in production build.
Decorators can be rewrite to work without using spy but I will have time
for this in several weeks only.
On Sat, Aug 18, 2018, 10:51 AM Jigar Gosar ***@***.***>
wrote:
> but spy was removed in v5
>
> I am not so sure since I am using spy in v5, let me know. Thanks.
> https://mobx.js.org/refguide/spy.html
>
> —
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub
> <
#16 (comment)
>,
> or mute the thread
> <
https://github.com/notifications/unsubscribe-auth/ACPAvRR7jL30yHm8Wc8k3XmtdpF4OT9Gks5uR8dlgaJpZM4WBFx6
>
> .
>
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#16 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AABOckdNH5or1GUMqVSyaD344UsNnd1Iks5uSo7xgaJpZM4WBFx6>
.
--
@jigargosar
+91 9892 006911
|
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
The text was updated successfully, but these errors were encountered: