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

fix: use different shellData in Form and Flow's ExtensionContext #3784

Merged
merged 2 commits into from
Aug 5, 2020

Conversation

yeze322
Copy link
Contributor

@yeze322 yeze322 commented Aug 5, 2020

Description

fixes #3783
refs #3721

Let Flow Editor consumes shellData from useShell('FlowEditor');
let Property Editor consumes shellData from useShell('PropertyEditor')

The difference is when an Action is selected, Property Editor's shellData should be the Action json, compared with Flow's data is whole dialog json.

image

Task Item

Screenshots

@coveralls
Copy link

coveralls commented Aug 5, 2020

Coverage Status

Coverage decreased (-0.004%) to 57.225% when pulling 9a15ea6 on yeze322:extension/separate-context into 65e4612 on microsoft:main.

@lei9444
Copy link
Contributor

lei9444 commented Aug 5, 2020

I have tested it and it works

@boydc2014 boydc2014 added the Approved to merge approved, waiting to be merged label Aug 5, 2020
@cwhitten cwhitten merged commit 78f7993 into microsoft:main Aug 5, 2020
lei9444 pushed a commit to lei9444/BotFramework-Composer-1 that referenced this pull request Jun 15, 2021
Co-authored-by: Dong Lei <donglei@microsoft.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Approved to merge approved, waiting to be merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

LgEditor shows wrong content and is not editable
5 participants