Cognigy / Cognigy-CLI

Cognigy-CLI is a series of tools meant to aid Cognigy.AI developers in maintaining local copies of their virtual agent projects.
Other
7 stars 6 forks source link

feat(knowledgeAI): Feature/47885 knowledgeAI to the cli #95

Closed XavierJordaMurria closed 1 year ago

XavierJordaMurria commented 1 year ago

To test this pr mainly test the extract and ingest functionality browse around the cli options "cognigy knowledge-ai -h" "cognigy knowledge-ai extract -h" "cognigy knowledge-ai ingest -h"

to see more details about the commands.

also you will need to have the config.json file up to date with a valid key

XavierJordaMurria commented 1 year ago

To test this pr mainly test the extract and ingest functionality browse around the cli options "cognigy knowledge-ai -h" "cognigy knowledge-ai extract -h" "cognigy knowledge-ai ingest -h"

to see more details about the commands.

also you will need to have the config.json file up to date with a valid key

kwinto commented 1 year ago

Was the command knowledge-aipart of the spec? I would change it to knowledge if not.

kwinto commented 1 year ago

./cognigy.js knowledge-ai create store --projectId 64b7f2e0416b00692704b90b --name "TestCLI" --language en-US results in error: unknown option '--name'

kwinto commented 1 year ago

./cognigy.js knowledge-ai create store --projectId 64b7f2e0416b00692704b90b --name "TestCLI" --language en-US results in error: unknown option '--name'

--name parameter is used in help example, we should fix the example usage

XavierJordaMurria commented 1 year ago

Was the command knowledge-aipart of the spec? I would change it to knowledge if not.

no it was not,

./cognigy.js knowledge-ai create store --projectId 64b7f2e0416b00692704b90b --name "TestCLI" --language en-US results in error: unknown option '--name'

yes, I've updated the help documentation already. Please use the following for creating stores: cognigy knowledge-ai create store "General Information" "General information about my business" --projectId 64a3d2aca915f8ce429c13ce --language en-US

kwinto commented 1 year ago

In this CLI, we have agent config parameter, which is also compulsory. Shouldn't we use it instead of --projectId parameter for knowledge commands?

kwinto commented 1 year ago

why is it still not approved?... /me hate microsoft for breaking github

XavierJordaMurria commented 8 months ago

:tada: This PR is included in version 1.5.0 :tada:

The release is available on:

Your semantic-release bot :package::rocket: