Skip to content
This repository has been archived by the owner on Jun 30, 2022. It is now read-only.

Commit

Permalink
Fix language argument missing in disconnect (#1510)
Browse files Browse the repository at this point in the history
  • Loading branch information
David Favretto authored and lzc850612 committed Jun 7, 2019
1 parent d56e3a9 commit ed41689
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 3 deletions.
1 change: 1 addition & 0 deletions lib/typescript/botskills/docs/connect-disconnect.md
Original file line number Diff line number Diff line change
Expand Up @@ -68,6 +68,7 @@ botskills disconnect [option]
| --ts | Determine your assistant project structure to be a TypeScript-like structure
| --noTrain | (OPTIONAL) Determine whether the skills connected are not going to be trained (by default they are trained) |
| --dispatchName [name] | (OPTIONAL) Name of your assistant's '.dispatch' file (defaults to the name displayed in your Cognitive Models file) |
| --language [language] | (OPTIONAL) Locale used for LUIS culture (defaults to 'en-us') |
| --dispatchFolder [path] | (OPTIONAL) Path to the folder containing your assistant's '.dispatch' file (defaults to './deployment/resources/dispatch/en' inside your assistant folder) |
| --outFolder [path] | (OPTIONAL) Path for any output file that may be generated (defaults to your assistant's root folder) |
| --lgOutFolder [path] | (OPTIONAL) Path for the LuisGen output (defaults to a 'service' folder inside your assistant's folder) |
Expand Down
4 changes: 1 addition & 3 deletions lib/typescript/botskills/src/botskills-disconnect.ts
Original file line number Diff line number Diff line change
Expand Up @@ -36,6 +36,7 @@ program
.option('--ts', 'Determine your assistant project structure to be a TypeScript-like structure')
.option('--noTrain', '[OPTIONAL] Determine whether the skills connected are not going to be trained (by default they are trained)')
.option('--dispatchName [name]', '[OPTIONAL] Name of your assistant\'s \'.dispatch\' file (defaults to the name displayed in your Cognitive Models file)')
.option('--language [language]', '[OPTIONAL] Locale used for LUIS culture (defaults to \'en-us\')')
.option('--dispatchFolder [path]', '[OPTIONAL] Path to the folder containing your assistant\'s \'.dispatch\' file (defaults to \'./deployment/resources/dispatch/en\' inside your assistant folder)')
.option('--outFolder [path]', '[OPTIONAL] Path for any output file that may be generated (defaults to your assistant\'s root folder)')
.option('--lgOutFolder [path]', '[OPTIONAL] Path for the LuisGen output (defaults to a \'service\' folder inside your assistant\'s folder)')
Expand Down Expand Up @@ -113,9 +114,6 @@ const language: string = args.language || 'en-us';
configuration.language = language;
const languageCode: string = (language.split('-'))[0];

// luisFolder validation
configuration.luisFolder = args.luisFolder || join(configuration.outFolder, 'Deployment', 'Resources', 'Skills', languageCode);

// dispatchFolder validation
configuration.dispatchFolder = args.dispatchFolder || join(configuration.outFolder, 'Deployment', 'Resources', 'Dispatch', languageCode);

Expand Down

0 comments on commit ed41689

Please sign in to comment.