Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow application to optionally parse messages before being displayed #29

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

timothy-janssen
Copy link

@timothy-janssen timothy-janssen commented Jan 16, 2019

Submitting for review.

Adding a hook that can be used by a hosting application to parse messages before they are sent to the user. The intention is to allow the Webchat to be able to communicate with the larger application it resides in. This behaves in much the same way as webchatMethods.getMemory, except gets called before messages are sent to the UI.

This article explains a simple use case: https://cai.tools.sap/blog/how-to-control-your-web-application-with-an-integrated-chatbot/

If applicationParse is an available method in the context of the application, it will be called before the messages are sent to setState and displayed to the user.
@timothy-janssen timothy-janssen changed the title Pull 1 Allow application to optionally parse messages before being displayed Mar 14, 2019
JWandrocke added a commit that referenced this pull request Apr 13, 2020
…_issue

Fixed an issue with Chrome scroll bar on windows 10 system, where the…
@cla-assistant
Copy link

cla-assistant bot commented Mar 7, 2023

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant