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

fix: load lu content need add system locale#3281

Merged
cwhitten merged 2 commits intomicrosoft:masterfrom
lei9444:publishluis
Jun 3, 2020
Merged

fix: load lu content need add system locale#3281
cwhitten merged 2 commits intomicrosoft:masterfrom
lei9444:publishluis

Conversation

@lei9444
Copy link
Contributor

@lei9444 lei9444 commented Jun 3, 2020

Description

The defaultLanguage in luis setting is used as fallback local. And fallbacklocal is used to set default recognizer when we use multi language.

When we load the lu content, the local is used to check the file local.
lubuild will check the filename, if the filename is 'a.en-us.lu', the local will set to 'en-us'.
If the file name is 'a.lu', lubuild will use the passed local.
But if the file is 'a.en-us.lu', and the local is 'it-it', the function will throw an error.

105ee385-30b9-4cb5-9497-757e0d911b01

Task Item

#minor
refs #3274

Screenshots

@coveralls
Copy link

coveralls commented Jun 3, 2020

Coverage Status

Coverage decreased (-0.001%) to 38.905% when pulling 1687168 on lei9444:publishluis into 0467744 on microsoft:master.

@a-b-r-o-w-n a-b-r-o-w-n changed the title fix: load lu content need add system local fix: load lu content need add system locale Jun 3, 2020
@a-b-r-o-w-n a-b-r-o-w-n added the Approved to merge approved, waiting to be merged label Jun 3, 2020
@cwhitten cwhitten merged commit 8c16531 into microsoft:master Jun 3, 2020
@lei9444 lei9444 deleted the publishluis branch February 1, 2021 02:05
lei9444 added a commit to lei9444/BotFramework-Composer-1 that referenced this pull request Jun 15, 2021
Co-authored-by: Chris Whitten <christopher.whitten@microsoft.com>
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.

4 participants