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: unify runtime path from ejecting #6520

Merged
merged 2 commits into from
Mar 24, 2021
Merged

Conversation

lei9444
Copy link
Contributor

@lei9444 lei9444 commented Mar 23, 2021

Description

issue
in 1.3.1 the eject runtime path is the relative path for the bot folder.
But in 1.4.0 the path is the relative path for the bot/settings.
So the bot created by 1.3.1 with custom runtime can't work.

fix
make the 1.3.1 and 1.4.0 use the same runtime path when do eject.
if the runtime is new runtime, still use the relative path for the bot/settings.

Task Item

#minor
closes #6519

Screenshots

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.008%) to 52.712% when pulling 245616e on lei9444:runtimepath into 294514f on microsoft:main.

@boydc2014 boydc2014 added the Approved to merge approved, waiting to be merged label Mar 23, 2021
@KarlXOL
Copy link

KarlXOL commented Mar 23, 2021

Thanks, this was helpful.
I've recreated the runtime in the meantime. This made bot creation and publishing work again.

Thanks
Karl

@boydc2014 boydc2014 merged commit 4dd99f4 into microsoft:main Mar 24, 2021
cwhitten pushed a commit that referenced this pull request Mar 31, 2021
@cwhitten cwhitten mentioned this pull request May 20, 2021
benbrown pushed a commit to benbrown/BotFramework-Composer that referenced this pull request May 24, 2021
benbrown pushed a commit that referenced this pull request Jun 11, 2021
lei9444 added a commit to lei9444/BotFramework-Composer-1 that referenced this pull request Jun 15, 2021
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.

Start bot failed - Upgrade to composer 4.0
4 participants