search

kubk / mobx-log

A logger + Redux devtools for Mobx 6+
MIT License
57 stars 7 forks source link

Redux devtools support #1

Closed kubk closed 2 years ago

kubk commented 3 years ago

Is it needed? Put :+1: if you think so :)

TechAkayy commented 2 years ago

Would be great to have redux devtools support, esp when the official Mobx devtools & Redux-remote both are unmaintained :-(

No point in reviving mobx devtools, as redux devtools is a better option as it could cover both redux & mobx..

Looks like redux-remote wasn't updated for the mobx later versions that used proxy, and no computed properties are now visible in redux-devtools, would be good to get that working. Thanks a bunch for asking :-)

Just because of this one limitation (not showing computed), I stopped using computed in mobx and rather I'm using verbose observable+autorun patterns, devtools is sooo important 👍

kubk commented 2 years ago

Hi @akkayy . What do you need the most from mobx-remotedev? Personally I've stopped to use remotedev because mobx-log covers all my needs. It logs store changes, you can have an access to store in browser console using store._STORENAME . So you can see observables, computeds and call actions

TechAkayy commented 2 years ago

Thanks for the quick response @kubk. Love your logger, will definitely use it as a supplement. Really sad that mobx that's such a smart one, has really no stable devtools.

Where devtools shine is when we use both mutable and immutable data in our state.. In large applications, we can end up with immutable nested objects of many levels like this:

image

This sort of data as computed, gets harder to track during dev on the console.

kubk commented 2 years ago

Hi @TechAkayy @dstaver The Redux devtools support has been added in 1.0.0

https://github.com/kubk/mobx-log/pull/33

TechAkayy commented 2 years ago

Wow, this is fantastic. Thanks bunch, will test it out.

TechAkayy commented 2 years ago

Thanks @kubk. Just started using v1.0.0 and it's great to have redux-devtools support. On that topic, I have added a related feature request here - https://github.com/kubk/mobx-log/issues/34. Thanks again!