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

Conversation

@srinaath
Copy link
Contributor

@srinaath srinaath commented Mar 13, 2020

This PR

  • Updates the package-lock files for Emulator patch version 4.8.1
  • Fixes a bug with restart conversation from activity that existed in 4.8.0. The issue being changes to restart status where not passed down from <Chat> to <OuterActivityWrapper>. Hence made use of <OuterActivityWrapperContainer> to funnel restartStatus and mode from state.chats[documentId] into activityWrapper which is a component supplied as middleware to Webchat for rendering an activity

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

Package lock updated

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

Changelog updated

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>
Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>
Copy link
Contributor

@tonyanziano tonyanziano left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@srinaath srinaath merged commit 4e714bf into v4.8.1 Mar 13, 2020
@srinaath srinaath deleted the srravich/bugfix/version-update-and-rendering-restart branch March 13, 2020 20:39
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 <srravich@microsoft.com>

* 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 <srravich@microsoft.com>

* 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.

3 participants