This repository has been archived by the owner on Dec 31, 2020. It is now read-only.
"export 'useComputed' was not found in 'mobx-react' #679
Comments
|
Thanks! 😄. |
I think import |
useLocalStore and useAsObservable source can be imported from mobx-react.
useObserver not (yet), as I don't see any clear benefit of it yet over
`observer` and `<Observer>`. It could tecnically be exposed as well without
any problem, I'm just not conviced yet that we should offer a third way of
introducing observability. I think it is would be just confusing.
Anyway, you could simply write your own `useObserver` (it's not exactly the
same but close enough): `const useObserver = fn =>
<Observer>{fn}</Observer>`.
If you would like to discuss useObserver further, please open a separate
issue since this one is about useComputed.
…On Thu, May 30, 2019 at 11:07 AM jackple ***@***.***> wrote:
import { useLocalStore, useObserver } from 'mobx-react-lite'
I think import useXXX from mobx-react-lite is weird beacuse I did not
install it by myself!!!
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#679?email_source=notifications&email_token=AAN4NBFZK5B3RFHEXQFT2DTPX6KOFA5CNFSM4HQ2LZKKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWRZXEA#issuecomment-497261456>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAN4NBDOETO36G7BKJRIA63PX6KOFANCNFSM4HQ2LZKA>
.
|
There is no |
Ah good point, they are present though, the typings seem to be outdated
indeed, will fix
…On Thu, May 30, 2019 at 11:44 AM jackple ***@***.***> wrote:
There is no useLocalStore or useAsObservable in mobx-react.d.ts
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#679?email_source=notifications&email_token=AAN4NBHVFBEVR4M3ATJ3NFTPX6OZLA5CNFSM4HQ2LZKKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWR4IEI#issuecomment-497271825>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAN4NBEFW4J5IXMN5TVFVZDPX6OZLANCNFSM4HQ2LZKA>
.
|
Fixed in 6.0.1
On Thu, May 30, 2019 at 11:47 AM Michel Weststrate <mweststrate@gmail.com>
wrote:
… Ah good point, they are present though, the typings seem to be outdated
indeed, will fix
On Thu, May 30, 2019 at 11:44 AM jackple ***@***.***> wrote:
> There is no useLocalStore or useAsObservable in mobx-react.d.ts
>
> —
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub
> <#679?email_source=notifications&email_token=AAN4NBHVFBEVR4M3ATJ3NFTPX6OZLA5CNFSM4HQ2LZKKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWR4IEI#issuecomment-497271825>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AAN4NBEFW4J5IXMN5TVFVZDPX6OZLANCNFSM4HQ2LZKA>
> .
>
|
mweststrate
added a commit
that referenced
this issue
May 30, 2019
Lol, edited the wrong file, 6.0.2 it is then.
Op do 30 mei 2019 12:03 schreef jackple <notifications@github.com>:
… @mweststrate <https://github.com/mweststrate> nothing changed!😄
https://github.com/mobxjs/mobx-react/blob/master/src/index.d.ts
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#679?email_source=notifications&email_token=AAN4NBEFRFSQ76VFR5CJOETPX6Q6DA5CNFSM4HQ2LZKKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWR5QGQ#issuecomment-497276954>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAN4NBHSGLDEBJI5AU3G7SLPX6Q6DANCNFSM4HQ2LZKA>
.
|
This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs or questions. |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
After mobx-react@6.x, I still need to import useComputed from 'mobx-react-lite'?
The text was updated successfully, but these errors were encountered: