Mobx: Typo in error message: "isObservable expects only 1 argument. Use isObsevableProp to inspect the observability of a property"

Created on 15 Mar 2018  路  2Comments  路  Source: mobxjs/mobx

https://github.com/mobxjs/mobx/blob/1090a2c80c68f459b572fe35cc4a1729cd4f897b/src/api/isobservable.ts#L39

and

https://github.com/mobxjs/mobx/blob/1090a2c80c68f459b572fe35cc4a1729cd4f897b/src/api/iscomputed.ts#L21

I tried copying and pasting isObsevableProp into my code after getting this message, but the function doesn't exist! Then I noticed the typo. It should be: isObservableProp

Also the docs need to be updated. They still say isObservable will take a 2nd arg.

馃悰 bug 馃摉 documentation

Most helpful comment

Hi @mnpenner,
thanks for the effort!

Are you willing to submit a PR?
I can help with anything related to PR submission, if you are new to it.

All 2 comments

Hi @mnpenner,
thanks for the effort!

Are you willing to submit a PR?
I can help with anything related to PR submission, if you are new to it.

Sorry I didn't get to it. A lot on my plate right now :smile: Thanks @mweststrate !

Was this page helpful?
0 / 5 - 0 ratings

Related issues

etinif picture etinif  路  3Comments

cafreeman picture cafreeman  路  4Comments

geohuz picture geohuz  路  3Comments

mdebbar picture mdebbar  路  3Comments

ansarizafar picture ansarizafar  路  4Comments