Skip to content
This repository has been archived by the owner on Jan 13, 2020. It is now read-only.

CookieManager.getAll failed #4

Closed
KingAmo opened this issue Dec 25, 2019 · 9 comments
Closed

CookieManager.getAll failed #4

KingAmo opened this issue Dec 25, 2019 · 9 comments
Labels
bug Something isn't working

Comments

@KingAmo
Copy link

KingAmo commented Dec 25, 2019

error log info:

ExceptionsManager.js:126 RNCookieManagerIOS.getAll was called with 1 arguments but
 expects 0 arguments. If you haven't changed this method yourself, this usually means
 that your versions of the native code and JavaScript code are out of sync. 
Updating both should make this error go away.
@KingAmo
Copy link
Author

KingAmo commented Dec 26, 2019

i invested in the source code and find this:
image
any help?

@safaiyeh safaiyeh added the bug Something isn't working label Dec 29, 2019
@safaiyeh
Copy link
Owner

@KingAmo Thanks for the issue.

Can you provide a code snippet on how you used this?

Thanks!

@safaiyeh
Copy link
Owner

safaiyeh commented Dec 29, 2019

@KingAmo could you test the branch in the PR and let me know if the exception is no longer thrown

@KingAmo
Copy link
Author

KingAmo commented Dec 30, 2019

    useEffect(() => {
        CookieManager.getAll()
            .then((res) => {
                console.log('CookieManager.getAll =>', res);
            });
    }, []);

the error still there in v3.2.3 , here is how i use it in iOS

@KingAmo
Copy link
Author

KingAmo commented Dec 30, 2019

error info:

ExceptionsManager.js:126 RNCookieManagerIOS.getAll was called with 1 arguments but
 expects 0 arguments. If you haven't changed this method yourself, this usually means 
that your versions of the native code and JavaScript code are out of sync. Updating 
both should make this error go away.

@safaiyeh
Copy link
Owner

Yeah it is not in v3.2.3, just on the branch ios-getAll-exception

@safaiyeh
Copy link
Owner

#10 is working for me locally, I'll merge it in and let you test it via npm

@safaiyeh
Copy link
Owner

Alrighty v3.2.4 is published, should fix this issue @KingAmo

@KingAmo
Copy link
Author

KingAmo commented Dec 31, 2019

@safaiyeh thanks a lot, it is fixed

@KingAmo KingAmo closed this as completed Dec 31, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants