Skip to content

Conversation

@germain-gg
Copy link
Contributor

@germain-gg germain-gg commented Mar 10, 2022

Related to element-hq/element-web#21215

  • Add check for stable/unstable thread server support
  • Use NamespacedValue for threads prefixes
  • determine server support before first sync
  • remove checks in server support has already been checked

This PR currently has no changelog labels, so will not be included in changelogs.

Add one of: T-Deprecation, T-Enhancement, T-Defect, T-Task to indicate what type of change this is plus X-Breaking-Change if it's a breaking change.

@codecov-commenter
Copy link

codecov-commenter commented Mar 10, 2022

Codecov Report

Merging #2228 (9da9d79) into develop (e16e7bc) will increase coverage by 0.13%.
The diff coverage is 73.91%.

@@             Coverage Diff             @@
##           develop    #2228      +/-   ##
===========================================
+ Coverage    57.68%   57.82%   +0.13%     
===========================================
  Files           90       90              
  Lines        16287    16300      +13     
  Branches      3752     3756       +4     
===========================================
+ Hits          9395     9425      +30     
+ Misses        6892     6875      -17     
Impacted Files Coverage Δ
src/filter.ts 75.00% <ø> (-1.20%) ⬇️
src/filter-component.ts 61.53% <21.42%> (-4.04%) ⬇️
src/client.ts 35.00% <92.30%> (+0.17%) ⬆️
src/NamespacedValue.ts 82.92% <100.00%> (+3.51%) ⬆️
src/models/event.ts 66.34% <100.00%> (+0.96%) ⬆️
src/models/thread.ts 55.26% <100.00%> (+17.24%) ⬆️
src/sync.ts 63.06% <0.00%> (-1.03%) ⬇️
src/models/relations.ts 37.33% <0.00%> (+0.66%) ⬆️
src/models/event-timeline-set.ts 77.99% <0.00%> (+0.77%) ⬆️
... and 2 more

Copy link
Member

@t3chguy t3chguy left a comment

Choose a reason for hiding this comment

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

LGTM otherwise

return this.stable;
}
return this.unstable;
}
Copy link
Member

Choose a reason for hiding this comment

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

maybe we should also override .altName to return the same as .name

@germain-gg germain-gg merged commit 9058dbf into develop Mar 11, 2022
@germain-gg germain-gg deleted the gsouquet/threads-stable-value branch March 11, 2022 09:04
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.

4 participants