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

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 <[email protected]>
Co-authored-by: Andy Brown <[email protected]>
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.

Multi language after deploy

6 participants