matrix-js-sdk
Comments attempting to explain the addEvent method
#3786
Merged

Comments attempting to explain the addEvent method #3786

andybalaam
andybalaam Warn when we drop an event trying to add it to a thread
ceb20766
andybalaam andybalaam added T-Task
andybalaam andybalaam requested a review 2 years ago
andybalaam andybalaam requested a review from dbkr dbkr 2 years ago
andybalaam andybalaam requested a review from richvdh richvdh 2 years ago
andybalaam andybalaam marked this pull request as draft 2 years ago
andybalaam Comment trying to explain lastEvent
5d0d9609
andybalaam Document some of my understanding of the addEvent logic
54998343
andybalaam andybalaam force pushed from 34a18652 to 54998343 2 years ago
andybalaam andybalaam changed the base branch from develop to andybalaam/warn-when-dropping-messages-in-addevent 2 years ago
andybalaam andybalaam marked this pull request as ready for review 2 years ago
Base automatically changed from andybalaam/warn-when-dropping-messages-in-addevent to develop 2 years ago
andybalaam Merge branch 'develop' into andybalaam/comments-in-addevent
503ec336
richvdh
richvdh approved these changes on 2023-10-09
andybalaam Refer to stable spec instead of MSC
ebfb0b31
andybalaam Re-word comments based on review
6a569bff
andybalaam andybalaam enabled auto-merge 2 years ago
andybalaam andybalaam merged f19f0a87 into develop 2 years ago
andybalaam andybalaam deleted the andybalaam/comments-in-addevent branch 2 years ago

Login to write a write a comment.

Login via GitHub

Reviewers
Assignees
No one assigned
Labels
Milestone