Skip to content
This repository has been archived by the owner on Oct 23, 2023. It is now read-only.

Ability to add custom Fulfillment text #96

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

Conversation

pierrefaure
Copy link

I need to add the tests but I'd need some directions about it.

I ran into a case where I needed both fulfillmentMessages and fulfillmentText so I thought submitting a PR would be a good idea.

@mattcarrollcode
Copy link
Contributor

mattcarrollcode commented Jul 17, 2018

Thanks for the PR. What is your use case for both and different fulfillment text and fulfillment messages?

@pierrefaure
Copy link
Author

We are doing this because we use Chatbase and our fulfillmentMessages is pretty complex to parse and might change in the future.

E.g. for now we have Text - Text - Payload - Text - Payload - Text

So by having fulfillmentText we can have a nice version of what we want to send to Chatbase for the transcript. + if we edit the order or add a new payload, it's way easier to just add the text we want to fulfillmentText.

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

Successfully merging this pull request may close these issues.

2 participants