Skip to content

Add messageEdited and messagedDeleted events to ChatAdapter. Update the unread message badge when a message is deleted.#3971

Merged
JamesBurnside merged 10 commits intomainfrom
jaburnsi/chat-deleted-unread-message-count
Jan 5, 2024
Merged

Add messageEdited and messagedDeleted events to ChatAdapter. Update the unread message badge when a message is deleted.#3971
JamesBurnside merged 10 commits intomainfrom
jaburnsi/chat-deleted-unread-message-count

Conversation

@JamesBurnside
Copy link
Copy Markdown
Member

What

  • Add messageEdited and messagedDeleted events to the ChatAdapter.
  • Use these events to update the unread message badge in the callwithchat composite when a message is deleted.

Why

Unread messages badge should update when an unread chat message is deleted.

How Tested

Verified locally deleting a chat messages decrements the unread message count to other participants in the callwithchat

Comment thread packages/communication-react/review/beta/communication-react.api.md Outdated
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jan 5, 2024

Chat bundle size is increased❗.

  • Current size: 1533934
  • Base size: 1533200
  • Diff size: 734

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jan 5, 2024

Calling bundle size is not changed.

  • Current size: 5621023
  • Base size: 5621023
  • Diff size: 0

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jan 5, 2024

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jan 5, 2024

CallWithChat bundle size is increased❗.

  • Current size: 6437810
  • Base size: 6436622
  • Diff size: 1188

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jan 5, 2024

Failed to pass the component examples UI Test. If this PR is for UI change and the error is snapshot mismatch, please add "update_snapshots" label to the PR for updating the snapshot.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jan 5, 2024

@azure/communication-react jest test coverage for stable.

Lines Statements Functions Branches
Base 21787 / 34490
63.16%
21787 / 34490
63.16%
594 / 1036
57.33%
1699 / 2715
62.57%
Current 21789 / 34490
63.17%
21789 / 34490
63.17%
594 / 1036
57.33%
1706 / 2718
62.76%
Diff 2 / 0
0.01%
2 / 0
0.01%
0 / 0
0%
7 / 3
0.19%

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jan 5, 2024

@azure/communication-react jest test coverage for beta.

Lines Statements Functions Branches
Base 44065 / 70815
62.22%
44065 / 70815
62.22%
914 / 1988
45.97%
2595 / 4208
61.66%
Current 44171 / 70941
62.26%
44171 / 70941
62.26%
914 / 1993
45.86%
2602 / 4221
61.64%
Diff 106 / 126
0.04%
106 / 126
0.04%
0 / 5
-0.11%
7 / 13
-0.02%

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jan 5, 2024

@JamesBurnside
Copy link
Copy Markdown
Member Author

Added breaking-change label as the breaking change is adding new string literals to the on function which is acceptable level

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jan 5, 2024

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jan 5, 2024

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jan 5, 2024

Failed to pass the UI Test. If this PR is for UI change and the error is snapshot mismatch, please add "update_snapshots" label to the PR for updating the snapshot.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jan 5, 2024

@JamesBurnside JamesBurnside enabled auto-merge (squash) January 5, 2024 19:39
@JamesBurnside JamesBurnside merged commit 909a9ec into main Jan 5, 2024
@JamesBurnside JamesBurnside deleted the jaburnsi/chat-deleted-unread-message-count branch January 5, 2024 19:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants