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

Add simple relations API #5186

Merged
merged 5 commits into from
May 16, 2019
Merged

Conversation

erikjohnston
Copy link
Member

This adds a simple relations API. Hopefully the commits make sense by themselves. Note that this is being PR'ed against a random base branch, as I don't want things to land on develop yet.

This implements matrix-org/matrix-spec-proposals#1849

@erikjohnston erikjohnston force-pushed the erikj/simple_pagination branch 2 times, most recently from cb457ee to 920f9b1 Compare May 14, 2019 16:27
@codecov
Copy link

codecov bot commented May 14, 2019

Codecov Report

❗ No coverage uploaded for pull request base (erikj/reactions_base@495e859). Click here to learn what that means.
The diff coverage is 88%.

@@                   Coverage Diff                   @@
##             erikj/reactions_base    #5186   +/-   ##
=======================================================
  Coverage                        ?   62.26%           
=======================================================
  Files                           ?      338           
  Lines                           ?    34786           
  Branches                        ?     5689           
=======================================================
  Hits                            ?    21661           
  Misses                          ?    11583           
  Partials                        ?     1542

@codecov
Copy link

codecov bot commented May 14, 2019

Codecov Report

❗ No coverage uploaded for pull request base (erikj/reactions_base@a9fc71c). Click here to learn what that means.
The diff coverage is 87.5%.

@@                   Coverage Diff                   @@
##             erikj/reactions_base    #5186   +/-   ##
=======================================================
  Coverage                        ?   62.27%           
=======================================================
  Files                           ?      338           
  Lines                           ?    34791           
  Branches                        ?     5687           
=======================================================
  Hits                            ?    21666           
  Misses                          ?    11583           
  Partials                        ?     1542

@erikjohnston erikjohnston requested a review from a team May 14, 2019 16:45
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 haven't read the MSC, but this seems generally plausible

synapse/rest/client/v2_alpha/relations.py Show resolved Hide resolved
synapse/storage/relations.py Outdated Show resolved Hide resolved
@erikjohnston erikjohnston merged commit f201a30 into erikj/reactions_base May 16, 2019
@erikjohnston erikjohnston deleted the erikj/simple_pagination 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