feat(chat): new branch for AI chat #66
Conversation
✅ Deploy Preview for carbon-for-ai ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
| </span> | ||
| </div> | ||
| </div> | ||
| <div class="test-section"> |
There was a problem hiding this comment.
This test section should only in the storybook code... if you want to change it to use controls you can see an example here how to use select: https://github.com/carbon-design-system/carbon-for-ibm-dotcom/blob/feat/cwc-storybook-7-vite/packages/carbon-web-components/src/components/tag/tag.stories.ts#L61 and storybook docs: https://storybook.js.org/docs/essentials/controls
| /** | ||
| * string denoting the bot name, default: 'bot' but can be changed to 'Watson' or 'client assistant' or any other name | ||
| */ | ||
| @property({ type: String, attribute: 'agentname' }) |
There was a problem hiding this comment.
| @property({ type: String, attribute: 'agentname' }) | |
| @property({ type: String, attribute: 'agent-name' }) |
usually we seperate two words by a dash in properties. Anywhere agentname is used should be updated to agent-name
| * string denoting the bot name, default: 'bot' but can be changed to 'Watson' or 'client assistant' or any other name | ||
| */ | ||
| @property({ type: String, attribute: 'agentname' }) | ||
| agentname; |
There was a problem hiding this comment.
| agentname; | |
| agentName; |
| * string denoting the unique behavior of the model designated by the user, appended to the private system prompt | ||
| */ | ||
| @property({ type: String, attribute: 'userprompt' }) | ||
| userprompt; |
There was a problem hiding this comment.
| userprompt; | |
| userPrompt; |
| /** | ||
| * string denoting the unique behavior of the model designated by the user, appended to the private system prompt | ||
| */ | ||
| @property({ type: String, attribute: 'userprompt' }) |
There was a problem hiding this comment.
| @property({ type: String, attribute: 'userprompt' }) | |
| @property({ type: String, attribute: 'user-prompt' }) |
same message as above with agent-name
| /** | ||
| * TEST OPTION FOR API CONNECTIONS | ||
| */ | ||
| private _selectedapi = 'local'; |
There was a problem hiding this comment.
| private _selectedapi = 'local'; | |
| private _selectedAPI = 'local'; |
There was a problem hiding this comment.
if this is only for testing can we move all of this into the storybook file chat.stories.ts
| min-block-size: 300px; | ||
| } | ||
|
|
||
| .chat-messages { |
There was a problem hiding this comment.
all of the classes should be in this similar format.#{$c4ai-prefix}--chat-container, so for .chat-messages, .user-message and so on they should look like .#{$c4ai-prefix}--chat-messages
| display: none; | ||
| } | ||
|
|
||
| .test-section { |
There was a problem hiding this comment.
if this is just for testing we can create a chat.scss inside __stories__
| // @ts-ignore | ||
| import styles from './chat.scss?inline'; | ||
|
|
||
| //const { stablePrefix: c4aiPrefix } = settings; |
There was a problem hiding this comment.
can probably uncomment this when you update the classes to use the prefix
Closes #
{{short description}}
Changelog
New
Changed
Removed
Testing / Reviewing
{{ Add descriptions, steps or a checklist for how reviewers can verify this PR works or not }}