-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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.
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
Updating cognitiveservices-luis-authoring SDK #4792
Updating cognitiveservices-luis-authoring SDK #4792
Conversation
…itiveServicesCredentials
sdk/cognitiveservices/cognitiveservices-luis-authoring/README.md
Outdated
Show resolved
Hide resolved
sdk/cognitiveservices/cognitiveservices-luis-authoring/src/lUISAuthoringClient.ts
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ahmedabuamra Some of the comments I made in #4791 applies here as well, so please take a look at those.
Can you update the PR description with list of changes that you had to do manually? Please do the same for the other PR as well
What I have changed:
|
sdk/cognitiveservices/cognitiveservices-luis-authoring/README.md
Outdated
Show resolved
Hide resolved
sdk/cognitiveservices/cognitiveservices-luis-authoring/README.md
Outdated
Show resolved
Hide resolved
sdk/cognitiveservices/cognitiveservices-luis-authoring/README.md
Outdated
Show resolved
Hide resolved
sdk/cognitiveservices/cognitiveservices-luis-authoring/package.json
Outdated
Show resolved
Hide resolved
sdk/cognitiveservices/cognitiveservices-luis-authoring/src/operations/apps.ts
Outdated
Show resolved
Hide resolved
sdk/cognitiveservices/cognitiveservices-luis-authoring/src/operations/azureAccounts.ts
Outdated
Show resolved
Hide resolved
sdk/cognitiveservices/cognitiveservices-luis-authoring/src/operations/examples.ts
Outdated
Show resolved
Hide resolved
sdk/cognitiveservices/cognitiveservices-luis-authoring/src/operations/features.ts
Outdated
Show resolved
Hide resolved
sdk/cognitiveservices/cognitiveservices-luis-authoring/src/operations/index.ts
Outdated
Show resolved
Hide resolved
sdk/cognitiveservices/cognitiveservices-luis-authoring/src/operations/model.ts
Outdated
Show resolved
Hide resolved
sdk/cognitiveservices/cognitiveservices-luis-authoring/src/operations/pattern.ts
Outdated
Show resolved
Hide resolved
sdk/cognitiveservices/cognitiveservices-luis-authoring/src/operations/permissions.ts
Outdated
Show resolved
Hide resolved
sdk/cognitiveservices/cognitiveservices-luis-authoring/src/operations/settings.ts
Outdated
Show resolved
Hide resolved
sdk/cognitiveservices/cognitiveservices-luis-authoring/src/operations/train.ts
Outdated
Show resolved
Hide resolved
sdk/cognitiveservices/cognitiveservices-luis-authoring/src/operations/versions.ts
Outdated
Show resolved
Hide resolved
sdk/cognitiveservices/cognitiveservices-luis-authoring/package.json
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@kayousef mentioned that we want to auto publish this library. Therefore, please update the package version and enable the autopublish feature by adding back the "autoPublish": true
in package.json file
@kayousef , @ramya-rao-a |
Since we have decided to get an update published with this PR approved, can we also get the browser sample updated? |
let authoringKey = process.env["luis-authoring-key"]; | ||
const creds = new CognitiveServicesCredentials(authoringKey); | ||
const region = "<your-region>"; | ||
const client = new LUISAuthoringClient( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
const client = new LUISAuthoringClient( | |
const client = new Azure.CognitiveservicesLuisAuthoring.LUISAuthoringClient( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ahmedabuamra This change is still pending.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
sorry I missed it.
sdk/cognitiveservices/cognitiveservices-luis-authoring/README.md
Outdated
Show resolved
Hide resolved
sdk/cognitiveservices/cognitiveservices-luis-authoring/README.md
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great!
All lines of code in this PR are auto-generated except updating the README.md file.