Chat: create storybook case for customized message #28662
Conversation
anna-shakhova
force-pushed
the
custom-chat-story
branch
2 times, most recently
from
e60a28a to
2fb624e
Compare
anna-shakhova
force-pushed
the
custom-chat-story
branch
from
2fb624e to
5314705
Compare
| height={height} | ||
| width={width} | ||
| items={items} | ||
| reloadOnChange={false} |
There was a problem hiding this comment.
We don't use DataSource object. I assume that this property doesn't matter for this sample.
| @@ -13,7 +13,8 @@ | |||
| "dependencies": { | |||
| "devextreme": "workspace:~", | |||
| "devextreme-react": "workspace:~", | |||
| "inferno": "^7.4.9" | |||
| "inferno": "^7.4.9", | |||
| "html-react-parser": "5.2.2" | |||
There was a problem hiding this comment.
| "html-react-parser": "5.2.2" | |
| "html-react-parser": "^5.2.2" |
|
|
||
| export const AIBotIntegration: Story = { | ||
| args: { | ||
| width: 500, |
There was a problem hiding this comment.
Let's omit dimensions options for this sample and pass them directly in Chat
| width, | ||
| height, | ||
| alerts, | ||
| }) => { |
There was a problem hiding this comment.
Are you sure the formatting here is correct? Please kindly check it
| }) => { | ||
| const [isProcessing, setIsProcessing] = useState(false); | ||
| const [assistantReplyIndex, setAssistantReplyIndex] = useState<number>(0); | ||
| const [copyBtnIcon, setCopyBtnIcon] = useState('copy'); |
There was a problem hiding this comment.
| const [copyBtnIcon, setCopyBtnIcon] = useState('copy'); | |
| const [copyButtonIcon, setCopyButtonIcon] = useState('copy'); |
| const repliesCount = assistantReplies.length; | ||
| const assistantReply = isProcessing ? regenerationMessage : assistantReplies[assistantReplyIndex % repliesCount]; | ||
|
|
||
| return [ userRequest, assistantReply ]; |
There was a problem hiding this comment.
| return [ userRequest, assistantReply ]; | |
| return [userRequest, assistantReply]; |
| </li> | ||
| </ul> | ||
| <p>AI continues to evolve, impacting nearly every aspect of modern life while raising | ||
| ethical, societal, and technological challenges.</p>`.replace(/\s{2,}/gm, ''), |
There was a problem hiding this comment.
Perhaps it's better to move this operation to the code so it applies to all input values and avoid duplication.
{HTMLReactParser(text.replace(/\s{2,}/gm, ''))}
What do you think?
| reloadOnChange={false} | ||
| showAvatar | ||
| showDayHeaders={false} | ||
| showMessageTimestamp |
There was a problem hiding this comment.
I suggest removing showAvatar, showMessageTimestamp, and showUserName since their default value is already true, and we don't override them
No description provided.