Skip to content

chore(*): Updated chat overview sample with new context...#1045

Open
gedinakova wants to merge 2 commits intovnextfrom
gedinakova/new-chat-overview-sample
Open

chore(*): Updated chat overview sample with new context...#1045
gedinakova wants to merge 2 commits intovnextfrom
gedinakova/new-chat-overview-sample

Conversation

@gedinakova
Copy link
Contributor

... as per George's suggestions.

: nothing;
};

private messageContentTemplate = (msg: any) => {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Shouldn't this be msg: IgcChatMessage? Prefer a type if available before any :)

@@ -83,26 +200,43 @@ export class ChatOverview {
public onMessageCreated = (e: CustomEvent) => {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

And this would be CustomEvent<IgcChatMessage> too, right?

@igdmdimitrov igdmdimitrov self-assigned this Sep 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants