matrix-js-sdk
Prevent threads code from making identical simultaneous API hits
#3541

Merged

Prevent threads code from making identical simultaneous API hits #3541

t3chguy merged 9 commits into develop from t3chguy/fix/25395

t3chguy added T-Enhancement

github-advanced-security commented on 2023-07-05

When building a thread from the original event onward consider the in…

66c8ebde

Set up a barrier around thread root fetching to prevent it happening …

f4529b2d

Update tests to skip all the superfluous API hits

c4cec91a

t3chguy force pushed from 4ae55f6f to c4cec91a 2 years ago

t3chguy changed the title ~~Consolidate simultaneous calls to GET API endpoints~~ Prevent threads code from making identical simultaneous API hits 2 years ago

t3chguy marked this pull request as ready for review 2 years ago

t3chguy requested a review 2 years ago

t3chguy requested a review from

germain-gg 2 years ago

t3chguy requested a review from

kerryarchibald 2 years ago

t3chguy enabled auto-merge 2 years ago

robintown commented on 2023-07-05

robintown removed review request from

germain-gg 2 years ago

robintown removed review request from

kerryarchibald 2 years ago

Add more defensive awaits

d3a32852

t3chguy requested a review from

robintown 2 years ago

robintown approved these changes on 2023-07-06

Fix replyCount

cb0841ff

Fix test assertion

54faae0b

Don't count thread root

eb824add

Track more threads stuff locally

ec8e516d

Fix tests

d2ebc339

t3chguy merged cd7c519d into develop 2 years ago

t3chguy deleted the t3chguy/fix/25395 branch 2 years ago

Reviewers

robintown

github-advanced-security

Assignees

No one assigned

Labels

T-Enhancement

Milestone

No milestone

matrix-js-sdk Prevent threads code from making identical simultaneous API hits #3541 Merged

Prevent threads code from making identical simultaneous API hits #3541

matrix-js-sdk
Prevent threads code from making identical simultaneous API hits
#3541

Merged