Skip to content
This repository has been archived by the owner on Jan 15, 2025. It is now read-only.

--sort option fails on bf luis:convert #1094

Closed
scheyal opened this issue Jan 20, 2021 · 0 comments · Fixed by #1115
Closed

--sort option fails on bf luis:convert #1094

scheyal opened this issue Jan 20, 2021 · 0 comments · Fixed by #1115
Assignees
Labels
needs-triage The issue has just been created and it has not been reviewed by the team. P1 Painful if we don't fix, won't block releasing
Milestone

Comments

@scheyal
Copy link
Contributor

scheyal commented Jan 20, 2021

Versions

c:>bf --version
@microsoft/botframework-cli/4.11.1 win32-x64 node-v12.18.4

Describe the bug

Run the bf luis:convert ... --sort command and get a failure. Same command works without --sort.

To Reproduce

>bf luis:convert --in test.json --out test.lu --sort
Unknown error during execution. Please file an issue on https://github.com/microsoft/botframework-cli/issues
Cannot read property 'sort' of undefined

>bf luis:convert --help
Convert .lu file(s) to a LUIS application JSON model or vice versa
 
USAGE
  $ bf luis:convert
 
OPTIONS
  -f, --force                    If --out flag is provided with the path to an existing file, overwrites that file
  -h, --help                     luis:convert help
  -i, --in=in                    Source .lu file(s) or LUIS application JSON model
  -o, --out=out                  Output file or folder name. If not specified stdout will be used as output
  -r, --recurse                  Indicates if sub-folders need to be considered to file .lu file(s)
  --culture=culture              Lang code for the LUIS application
  --description=description      Text describing the LUIS applicaion
  --log                          Enables log messages
  --name=name                    Name of the LUIS application
  --schemaversion=schemaversion  Schema version of the LUIS application
  --sort                         When set, intent, utterances, entities are alphabetically sorted in .lu files
  --versionid=versionid          Version ID of the LUIS application

Expected behavior

Give a clear and concise description of what you expected to happen.

Screenshots

If applicable, add screenshots to help explain your problem.

Additional context

Add any other context about the problem here.

[bug]

@scheyal scheyal added the needs-triage The issue has just been created and it has not been reviewed by the team. label Jan 20, 2021
@scheyal scheyal added this to the R12 milestone Jan 20, 2021
@munozemilio munozemilio added the P1 Painful if we don't fix, won't block releasing label Feb 8, 2021
@munozemilio munozemilio self-assigned this Feb 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
needs-triage The issue has just been created and it has not been reviewed by the team. P1 Painful if we don't fix, won't block releasing
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants