Update to match changes in streaming extensions module, fix build issues...#190
Merged
ckkashyap merged 1 commit intomicrosoft:ckk/protocoljsfrom Aug 26, 2019
Conversation
…ues, add excludes to webpack config
ckkashyap
pushed a commit
that referenced
this pull request
Sep 13, 2019
…ues, add excludes to webpack config (#190)
cwhitten
added a commit
that referenced
this pull request
Mar 4, 2020
* Added dependency to protocol * Start conversation is sent. * Connection works * DLJS Works a little bit ... * Working version * Make old and new work together * Cleanup * minor refactoring * Fixed the dependency on protocol * Do not use conversation id during connect * Support posting of attachments over streaming protocol * Minor correction. * Make the attachments inline. * Incorporate name change * Remove old protocol dependencies * Update to match changes in streaming extensions module, fix build issues, add excludes to webpack config (#190) * With tests * Add CI build * Use BotBuilder-JS with PR #1132 * Use later build * Update TGZ * Add Git tag * Update build script * Update script * Enable tests * Add CONTRIBUTING.md * Verbage * Fix for streaming post activity test. Ths test is still commented out because there is an issue in ASE that prevents the test from passing. The fix was validated by running it against DirectLineFlex 2884ae1c356971545b0580297619edb4d610b6cb * post activit works * Added conversationUpdate test * fixed streaming extensions library casing * Added the right tgz * addressed review comment * Fixed a couple of attachment bugs * Commenting out streaming tests to see if it makes Travis happy * Fixed minor bug. * New streaming extensions library * All tests pass * Valdiate the attachment content * Changed the contenttype field to type to match the change in the streaming extensions library and also set the content type of the first stream while uploading attachments to application/vnd.microsoft.activity. * Add a newer webpack * receive attachment test * Extracted out the streaming logic into another file * retry works * removed the need for ActivityGroup * new connect * cleaned up * cleanup * post error is handled * formatting * conversation id and referencegrammar set * retry added * remove unused code * added token refresh * refresh updated * refresh retry * referencegrammar set * js equal stuff * import reordering * remove extra blank line * directline fixed * All tests pass with the C# bot * right streaming tgz * fixed test * commented out the attachment test * fixed the attachment order bug * add my username * some refactoring * minor refactoring * Change the dependency to npm * Updated the package-lock.json * Fix failing test * Addressed some review comments. * remove commented out line * Addressed some review comments * Addressed more comments. * More review comments addressed * Addressed more comments. * Addressed more comments. * Addressed more comments. * Addressed more comments * Changed the custructor option * Addressed more comments. * Added support for IE11 * Switched to using fetch * Refresh token uses fetch now * Moved the connect logic away from rxjs * Addressed review comments * Point to preview version of botframework-streaming on npm Co-authored-by: ckkMicrosoft <30480427+ckkMicrosoft@users.noreply.github.com> Co-authored-by: Daniel Evans <daniel@ddevans.com> Co-authored-by: William Wong <compulim@users.noreply.github.com>
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
...add excludes to webpack config