search

GoogleCloudPlatform / dialogflow-integrations

Dialogflow integrations with multiple platforms including KIK, Skype, Spark, Twlio, Twitter and Viber
https://cloud.google.com/dialogflow/
Apache License 2.0
247 stars 499 forks source link

Fix race condition when deleting webhooks and potential error when calling detectIntent #146

Closed leulaf closed 2 years ago

leulaf commented 2 years ago

Fixed race conditon when deleting webhooks. Also when the ES integration was @ed to posts it was calling detectIntent with the entire text including the @, so detectIntent was receiving for example "@CXBot hello" instead of just "Hello". I have implemented the same solution that I did for the twitter CX integration which was to replace the @ + screenName with an empty string.

chihhsuan-google commented 2 years ago

Leul, thanks for the fix!