Skip to content
New issue

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

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

Already on GitHub? # to your account

Add inspector support to protocolHandler #1617

Merged
merged 2 commits into from
Jun 4, 2019
Merged

Add inspector support to protocolHandler #1617

merged 2 commits into from
Jun 4, 2019

Conversation

stevengum
Copy link
Member

Fixes #1567

Uses feedback on #1601

Changes

  • Add support for bfemulator://inspector.open
    • Open emulator in debug mode
    • Open emulator and connect to bot in debug mode
    • Add test for inspector.open, refactor test for livechat.open

Copy link
Contributor

@justinwilaby justinwilaby left a comment

Choose a reason for hiding this comment

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

LGTM. :shipit:

@justinwilaby
Copy link
Contributor

I can confirm this feature works on a distribution build of the Emulator for Mac:
Screen Shot 2019-06-04 at 3 10 17 PM

@tonyanziano
Copy link
Contributor

Verified as well on Windows. 👍

@stevengum stevengum merged commit d1c6efe into master Jun 4, 2019
@stevengum stevengum deleted the protocol-update branch June 4, 2019 23:58
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bot Inspector] Open a inspector mode conversation via protocol
3 participants