Skip to content

[IMPROVEMENT] Response message read order handling#79

Merged
ear-dev merged 5 commits intocx_developmentfrom
response_message_read_order_handling
Jul 21, 2021
Merged

[IMPROVEMENT] Response message read order handling#79
ear-dev merged 5 commits intocx_developmentfrom
response_message_read_order_handling

Conversation

@AlexanderKanakis
Copy link
Copy Markdown
Collaborator

Ensures that all text messages are read and sent to client before custom payloads and other fulfillment dialog options.

@stetak
Copy link
Copy Markdown

stetak commented Jul 15, 2021

I still messages being dropped. Room id g6WigbXuKA3PkkgBN on https://ba.messaging.vega-preprod.viasat.io/omnichannel/current - The message dropped before the boleto link was : "You can download your boleto below."

@AlexanderKanakis @ear-dev

@ear-dev
Copy link
Copy Markdown

ear-dev commented Jul 15, 2021

@AlexanderKanakis do you think you can repro the issue that @stetak is seeing?

@AlexanderKanakis
Copy link
Copy Markdown
Collaborator Author

@stetak @AlexanderKanakis Dialogflow merged the all custom payloads with the last text message of the response. It worked in the context of the previous custom payloads (customFields), but the mediaCard needed to be separated. It should be fixed now

@ear-dev
Copy link
Copy Markdown

ear-dev commented Jul 15, 2021

Thanks @AlexanderKanakis I'll deploy and test this tomorrow

@ear-dev
Copy link
Copy Markdown

ear-dev commented Jul 16, 2021

@AlexanderKanakis We tested this and it's fixed! Awesome, thanks......

@bhardwajaditya can you review this one please? Just take a quick look and if it looks good I'll merge... thanks!

@ear-dev ear-dev merged commit 42371f0 into cx_development Jul 21, 2021
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.

4 participants