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

feat: Added network and activity logs to Web Chat log panel #6638

Merged
merged 62 commits into from
Apr 7, 2021

Conversation

tonyanziano
Copy link
Contributor

@tonyanziano tonyanziano commented Apr 1, 2021

Description

This PR adds network and Web Chat activity logs to the Web Chat log panel.

Task Item

Fixes #6511

image

image

@tonyanziano tonyanziano marked this pull request as ready for review April 1, 2021 22:58
@tonyanziano tonyanziano changed the title Added network and activity logs to Web Chat log panel feat: Added network and activity logs to Web Chat log panel Apr 1, 2021
srinaath
srinaath previously approved these changes Apr 6, 2021
@tonyanziano tonyanziano merged commit 6dada46 into main Apr 7, 2021
@tonyanziano tonyanziano deleted the toanzian/web-chat-log branch April 7, 2021 02:15
@cwhitten cwhitten mentioned this pull request May 20, 2021
lei9444 pushed a commit to lei9444/BotFramework-Composer-1 that referenced this pull request Jun 15, 2021
…t#6638)

* Added fadedWhenReadOnly option to JsonEditor

* Added activity tracking and inspection panel

* Plumbed data through to Web Chat inspection panel.

* More style polish

* Made inspector resizable and did some renames

* Plumbed network response data into log

* Fixed logging network responses to client

* Condensed network and activity traffic into one socket

* Consolidated network errors into traffic channel

* Cleaned up types and recoil code

* Factored out client code into smaller components

* Only show error indicator when unread errors are present

* Fixed client tests

* Fixed server tests.

* Removing old name directory

* Adding back properly named dir

* PR comments

* Fixed timestamps

* More PR comments addressed

* Fixed tests

* Addressed more PR comments

* Fixed types

* Added test for logNetworkTraffic

* Fixed test.

* Updated recoil dispatcher names to be consistent.

* Added web chat state tests

* Adjusted fonts in both log and inspector panes to match

* Merge conflicts resolved

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

* Smoothing out merge conflict

* Moved network logging middleware to specific routes

* Fixed attachment route tests.

Co-authored-by: Ben Yackley <[email protected]>
Co-authored-by: Srinaath Ravichandran <[email protected]>
Co-authored-by: Dong Lei <[email protected]>
Co-authored-by: Srinaath Ravichandran <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support activity and bot state inspection in the embedded Web Chat control
6 participants