Skip to content
This repository was archived by the owner on Jul 9, 2025. It is now read-only.

Comments

fix: open bot read property 'sdk' is undefined#3016

Merged
boydc2014 merged 1 commit intomasterfrom
zhixzhan/route-fix
May 13, 2020
Merged

fix: open bot read property 'sdk' is undefined#3016
boydc2014 merged 1 commit intomasterfrom
zhixzhan/route-fix

Conversation

@zhixzhan
Copy link
Contributor

@zhixzhan zhixzhan commented May 13, 2020

Description

seems imported by #2922 (not causing this bug, but exposed bug in state), which visual editor call navigate methods with empty projectId, dialogId.

this fix works as a hot fix, but dig deep, the bug reason comes from state.designPageLocation not sync with state. further fix tracked via #3022

Task Item

close #3015

Screenshots

@github-actions
Copy link

Coverage Status

Coverage remained the same at 0.0% when pulling 8b36164 on zhixzhan/route-fix into 8558911 on master.

@boydc2014 boydc2014 added the Approved to merge approved, waiting to be merged label May 13, 2020
@boydc2014 boydc2014 merged commit 08b8579 into master May 13, 2020
@boydc2014 boydc2014 deleted the zhixzhan/route-fix branch May 13, 2020 08:22
lei9444 pushed a commit to lei9444/BotFramework-Composer-1 that referenced this pull request Jun 15, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Approved to merge approved, waiting to be merged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants