Skip to content
This repository was archived by the owner on Jan 5, 2026. It is now read-only.

Added tests to ensure restart bubble not present#2107

Merged
srinaath merged 2 commits intov4.8.1from
srravich/bugfix/speech-bot-restart
Mar 13, 2020
Merged

Added tests to ensure restart bubble not present#2107
srinaath merged 2 commits intov4.8.1from
srravich/bugfix/speech-bot-restart

Conversation

@srinaath
Copy link
Copy Markdown
Contributor

This PR ensures "Restart Conversation from here" bubble does not appear for DL Speech bots. This functionality is not available for DL speech bots yet.

Signed-off-by: Srinaath Ravichandran <[email protected]>

Should hide restart bubble if speech bot

Signed-off-by: Srinaath Ravichandran <[email protected]>

Typo fixed

Signed-off-by: Srinaath Ravichandran <[email protected]>
Signed-off-by: Srinaath Ravichandran <[email protected]>
@srinaath srinaath merged commit a3f7887 into v4.8.1 Mar 13, 2020
@tonyanziano tonyanziano deleted the srravich/bugfix/speech-bot-restart branch March 18, 2020 18:19
srinaath pushed a commit that referenced this pull request Mar 18, 2020
* Fixed missing item in File menu on Win / Lin. (#2100)

* Fixed missing item in File menu on Win / Lin.

* Fixed changelog entry PR #

* Bump version to 4.8.0 (#2101)

* Package updated

Signed-off-by: Srinaath Ravichandran <[email protected]>

* Added missing icon file for Linux / Mac (#2104)

* Prevent passing props to activity middleware (#2105)

* Prevent passing props to activity middleware

* Package lock updated

* Changelog updated

* Corrected changelog

* Added tests to ensure restart bubble not present (#2107)

Should hide restart bubble if speech bot

Co-authored-by: Srinaath Ravichandran <[email protected]>

* Fixed a bug that was causing a blank User 'id' field. (#2108)

* Fixed a bug that was causing a blank User 'id' field.

* Added extra assertion in tests.

* Ngrok Startup Options (#2111)

* Make sure ngrok running loop not hit everytime

* Ngrok reporting made accurate (#2113)

* Recator isUsingNgrok
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