prepare 2.17.1 release #204
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
update some test dependencies to avoid vulnerability warnings
* Initial commit * Initial commit * Added unit test * Example server bootstrap implementation * Updated packages and exclude server bootstrap code. * Update package-lock.json * Added more tests. * Moved state initialisation into constructor.
pin Node version to 11.10.1 due to Jest problem
add support for optional trackReasons flag for experimentation
replace rm with rimraf in react wrapper build
add azure pipeline
# Conflicts: # package-lock.json # package.json
…ents add image-loading event delivery logic factored out of js-sdk-common
use js-sdk-common 3.1.1 for event payload ID fix
use js-sdk-common 3.1.2 and loosen our other dependency
# Conflicts: # package-lock.json # package.json
enable diagnostic events in JS SDK
standardize linting
bwoskow-ld
approved these changes
Mar 7, 2020
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
[2.17.1] - 2020-03-06
Fixed:
waitForInitialization()and provided an error handler for the promise that was returned by that method. While that is correct behavior if the application did callwaitForInitialization()(any promise that might be rejected should have an error handler attached), it is inappropriate if the application did not callwaitForInitialization()at all-- which is not mandatory, since the application could use events instead, orwaitUntilReady(), or might simply not care about waiting for initialization. This has been fixed so that no such promise is created until the first time the application callswaitForInitialization(); subsequent calls to the same method will return the same promise (since initialization can only happen once).onoroffwhile handling an event. This has been fixed so that all event handlers that were defined at the time the event was fired will be called; any changes made will not take effect until the next event.