Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Merged
merged 2 commits into from
Mar 16, 2020

Conversation

tonyanziano
Copy link
Contributor

#2106

===

Fixed some faulty logic that was not behaving as expected.

The select() effect from redux-saga returns an object, so it is always truthy no matter what and will resolve to the yielded value.

Ex:

const val = yield select(() => false) || 'some other value';
console.log(val);  // false

So even when the custom user ID was set to the empty string, the start conversation logic was always initializing the conversation with the blank ID rather than defaulting to a freshly generated GUID.

@tonyanziano tonyanziano merged commit 6a43fda into v4.8.1 Mar 16, 2020
@tonyanziano tonyanziano deleted the toanzian/id-fix branch March 16, 2020 19:55
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 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.

2 participants