Error handling

[mediumTaj]

Summary

Refactoring SDK to have a success and failure callbacks. We also now pass a dictionary for custom data.

We can now use generic handlers as callbacks for multiple response types. The raw json response is always passed as json in the customData dictionary.

void Start()
{
    //  Call with generic callbacks
    conversation.Message(OnSuccess, OnFail, _workspaceId, "");
    discovery.GetEnvironments(OnSuccess, OnFail);

    //  Call with sepcific callbacks
    conversation.Message(OnMessage, OnGetEnvironmentsFail, _workspaceId, "");
    discovery.GetEnvironments(OnGetEnvironments, OnMessageFail);
}

//  Generic success callback
private void OnSuccess<T>(T resp, Dictionary<string, object> customData)
{
    Log.Debug("ExampleCallback.OnSuccess()", "Response received: {0}", customData["json"].ToString());
}

//  Generic fail callback
private void OnFail(RESTConnector.Error error, Dictionary<string, object> customData)
{
    Log.Error("ExampleCallback.OnFail()", "Error received: {0}", error.ToString());
}

//  OnMessage callback
private void OnMessage(object resp, Dictionary<string, object> customData)
{
    Log.Debug("ExampleCallback.OnMessage()", "Response received: {0}", customData["json"].ToString());
}

//  OnGetEnvironments callback
private void OnGetEnvironments(GetEnvironmentsResponse resp, Dictionary<string, object> customData)
{
    Log.Debug("ExampleCallback.OnGetEnvironments()", "Response received: {0}", customData["json"].ToString());
}

//  OnMessageFail callback
private void OnMessageFail(RESTConnector.Error error, Dictionary<string, object> customData)
{
    Log.Error("ExampleCallback.OnMessageFail()", "Error received: {0}", error.ToString());
}

//  OnGetEnvironmentsFail callback
private void OnGetEnvironmentsFail(RESTConnector.Error error, Dictionary<string, object> customData)
{
    Log.Error("ExampleCallback.OnGetEnvironmentsFail()", "Error received: {0}", error.ToString());
}

[mediumTaj]

@kimberlysiva Can you test this when you have a chance?

[kimberlysiva]

@mediumTaj Thanks so much for this! The failure callbacks are great, I'm correctly catching the 409 errors for STT custom language models now.

I've tested STT and Conversation on Mac, Windows, and Hololens. Everything is working great so far!