Skip to content
This repository has been archived by the owner on Dec 7, 2022. It is now read-only.

✨ Add documents for bot types and medallia #1447

Merged

Conversation

ahuppert
Copy link
Contributor

Work in Progress - do not merge


#### Description
Parsing the response of the Madallia Adapter failed.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
Parsing the response of the Madallia Adapter failed.
Parsing the response of the Medallia Adapter failed.

In this case the Medallia Adapter sent some data we did not expect.
#### Solutions
Contact the Medallia Adapter and provide them the data of the conversation error.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How should a customer know how to contact "the Medallia adapter"? Check with Dong how they should be contacted in this case.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

changed it now to support for now until we have proper solution path

<td>Workspace URL</td>
<td>Watson Assistant Workspace URL. Note that this workspace URL changes according to your account. In order to figure out which URL is correct for you, check the location of your Assistant in IBM's Cloud console. Then, choose the corresponding URL based on geo-location. For example, if your IBM location is "eu-gb", the London URL is the correct one for you.</td>
<td>https://api.eu-de.assistant.watson.cloud.ibm.com</td>
<td>Client ID</td>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sure that his should all go into ibm-watson-assistant-version-2.md ?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

no :) will fix it

yarn.lock Outdated
@@ -0,0 +1,3549 @@
# THIS IS AN AUTOGENERATED FILE. DO NOT EDIT THIS FILE DIRECTLY.
# yarn lockfile v1
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

did you add this file on purpose?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

no. apparently yarn was never used before
will remove it

@snikic
Copy link
Contributor

snikic commented Jan 24, 2022

@ahuppert There is also a staging branch where we could first commit to and review the changes:
https://github.com/LivePersonInc/developers-community#environments
Lets do that after you worked on my changs and also get Ektas feebdback

@itay1313 itay1313 merged commit 1bc1688 into LivePersonInc:master Feb 7, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants