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.

Sign up for GitHub

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

Jump to bottom

enabledForAllModels missing in phraselist object #327

Closed
munozemilio opened this issue Nov 1, 2019 · 1 comment · Fixed by #366
Closed

enabledForAllModels missing in phraselist object #327

munozemilio opened this issue Nov 1, 2019 · 1 comment · Fixed by #366
Assignees
@vishwacsena
Labels
lu non luis api commands P0 Must Fix. Release-blocker R7 Release 7 - December 10th, 2019

Comments

@munozemilio
Copy link
Member

When a phraselist is not added as a feature the enabledForAllModels property is missing when going: luis.json -> lu -> luis.json.

The expecte behavior is to set as true if not added to any intent.
@vishwacsena vishwacsena added the lu non luis api commands label Nov 8, 2019
@vishwacsena vishwacsena self-assigned this Nov 19, 2019
@vishwacsena
Copy link
Contributor

  • provide control for developers to set if a phrase list is enabled for all models or not
  • provide control for developers to enable/ disable a phrase list (controls activated property)

@vishwacsena vishwacsena added P0 Must Fix. Release-blocker R7 Release 7 - December 10th, 2019 labels Nov 19, 2019
@vishwacsena vishwacsena mentioned this issue Nov 20, 2019
Merged
@snyk-bot snyk-bot mentioned this issue Dec 2, 2021
Open
@baby636 baby636 mentioned this issue May 14, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@vishwacsena vishwacsena

Labels
lu non luis api commands P0 Must Fix. Release-blocker R7 Release 7 - December 10th, 2019
Projects
None yet
Milestone
No milestone
2 participants
@vishwacsena @munozemilio