Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

Add relation aggregation APIs #5192

Merged
merged 6 commits into from
May 16, 2019

Conversation

erikjohnston
Copy link
Member

@erikjohnston erikjohnston commented May 15, 2019

Builds on top of #5186. Does rely on #5191, but that has already been merged in the base branch.

The actual bundling of relations into events is disabled by default, but can be enabled via the undocumented experimental_msc1849_support_enabled config flag.

@erikjohnston erikjohnston force-pushed the erikj/relations_aggregations branch 2 times, most recently from 1e0ee71 to 467678e Compare May 15, 2019 14:29
@codecov
Copy link

codecov bot commented May 15, 2019

Codecov Report

Merging #5192 into erikj/reactions_base will increase coverage by 0.13%.
The diff coverage is 86.44%.

@@                   Coverage Diff                   @@
##           erikj/reactions_base   #5192      +/-   ##
=======================================================
+ Coverage                 62.27%   62.4%   +0.13%     
=======================================================
  Files                       338     338              
  Lines                     34791   34955     +164     
  Branches                   5687    5713      +26     
=======================================================
+ Hits                      21666   21814     +148     
- Misses                    11583   11595      +12     
- Partials                   1542    1546       +4

@erikjohnston erikjohnston requested a review from a team May 16, 2019 09:55
@erikjohnston erikjohnston mentioned this pull request May 16, 2019
2 tasks
Copy link
Member

@richvdh richvdh left a comment

Choose a reason for hiding this comment

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

I've kinda skimmed this tbh, but it looks generally sane. A few nits.

synapse/rest/client/v2_alpha/relations.py Show resolved Hide resolved
synapse/rest/client/v2_alpha/relations.py Outdated Show resolved Hide resolved
synapse/rest/client/v2_alpha/relations.py Outdated Show resolved Hide resolved
synapse/storage/relations.py Show resolved Hide resolved
synapse/storage/relations.py Show resolved Hide resolved
synapse/storage/relations.py Show resolved Hide resolved
@erikjohnston erikjohnston merged commit 5c39d26 into erikj/reactions_base May 16, 2019
@erikjohnston erikjohnston deleted the erikj/relations_aggregations branch January 9, 2020 15:51
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants