Skip to content

fix(chat): elaborate the bidirectionality of the messages array#438

Open
ChronosSF wants to merge 1 commit into
vnextfrom
sstoychev/elaborate-chat-message-usage
Open

fix(chat): elaborate the bidirectionality of the messages array#438
ChronosSF wants to merge 1 commit into
vnextfrom
sstoychev/elaborate-chat-message-usage

Conversation

@ChronosSF

Copy link
Copy Markdown
Member
  • Documented messages as a bidirectional collection.
  • [Angular] Clarified that Chat mutates the bound collection in place, rather than supporting Angular banana-in-a-box two-way binding syntax.
  • Warned against appending the message again in messageCreated.
  • Added an array-spread example to preserve the original collection.
  • Explained that the shallow copy still shares existing message objects.

@ChronosSF
ChronosSF requested a review from gedinakova July 16, 2026 15:57
@gedinakova gedinakova added the ✅ status: verified Applies to PRs that have passed manual verification label Jul 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

✅ status: verified Applies to PRs that have passed manual verification

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants