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: language err for build luis in azure deploy #4155

Merged
merged 5 commits into from
Sep 18, 2020

Conversation

alanlong9278
Copy link
Contributor

@alanlong9278 alanlong9278 commented Sep 16, 2020

Description

Bug fix: default language for build luis in azure deploy

In composer settings, we use defaultLanguge as the fallback language for both LU and LG.
But currently there is a bug in the azure publish plugin that it takes the settings.language as the default language for lubuild.

This PR is to fix that issue.

Task Item

closes #4136

Screenshots

@coveralls
Copy link

coveralls commented Sep 16, 2020

Coverage Status

Coverage remained the same at 55.313% when pulling 66d1371 on julong/language-err-for-publish into f4853c6 on main.

@luhan2017 luhan2017 added the Approved to merge approved, waiting to be merged label Sep 18, 2020
@cwhitten cwhitten merged commit 9ba5e84 into main Sep 18, 2020
@cwhitten cwhitten deleted the julong/language-err-for-publish branch September 18, 2020 18:10
alanlong9278 added a commit to alanlong9278/BotFramework-Composer that referenced this pull request Sep 21, 2020
* main:
  fix: language err for build luis in azure deploy (microsoft#4155)
  fix: Unify skill and skill setting dispatchers (microsoft#4173)
@cwhitten cwhitten mentioned this pull request Nov 13, 2020
lei9444 pushed a commit to lei9444/BotFramework-Composer-1 that referenced this pull request Jun 15, 2021
* language err for build luis

* refactor

Co-authored-by: Chris Whitten <christopher.whitten@microsoft.com>
Co-authored-by: Andy Brown <asbrown002@gmail.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.

Multi language after deploy
5 participants