Response from proxy.invoke promise not getting returned about react-native-signalr HOT 9 CLOSED

Hmm, I'll look into this tonight!

from react-native-signalr.

@chris-gregg Sry for taking such a long time before looking at this. But it think that what you are receiving in the done-callback is correct. If you want to receive the actual message sent from the server you should set up a listener before calling invoke like this:

        proxy.on('responseFromServer', (response) => {
           console.log("FROM SERVER", response);
        });
        let messagePromise = proxy.invoke('messageToServer', 'hej');
        messagePromise.done((response) => {
            console.log ("response", response');
        }).fail(function (error) {
            console.log('Invocation of NewContosoChatMessage failed. Error: ' + error);
        });

I have not looked at if the signalr-client used in a browser does the same, but I expect it does. The done for the message is only a status about that particular message.

I will closed this for now.

from react-native-signalr.

Hey @olofd, thanks for taking a look. With SignalR you can receive data from the server via a listener/callback as you suggested, but should also be able to access a response from the server via the promise.

I was going to take a look and submit a pull request, but noticed in your last commit that you actually fixed the problem! Looks like the issue was that resolveWith was originally returning the proxy, and not the data (response).

So I can now do this, and successfully retrieve the response:

let messagePromise = proxy.invoke('messageToServer', 'hey');
messagePromise.then((response) => {
     console.log ('response', response);
}).fail(function (error) {
     console.log('Invocation of NewContosoChatMessage failed. Error: ' + error);
});

Thanks again!

from react-native-signalr.

ah! Great! Sry for the incorrect answer before! Let me know if there are any other issues with the lib!

from react-native-signalr.

Will do, looking forward to using this!

from react-native-signalr.

@chris-gregg Can you take a look at this:
#2

I think there should be an apply there, elsewise I get my data wrapped in an array.
Have you experienced this?

from react-native-signalr.

Yeah, you're right. I'm seeing the response encapsulated in an array too.

from react-native-signalr.

Great! Thanks for the confirmation. I'll merge and release a new version then.

from react-native-signalr.

Release 0.0.6

from react-native-signalr.