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

Remove support for aggregating reactions #15172

Merged
merged 3 commits into from Feb 28, 2023
Merged

Remove support for aggregating reactions #15172

merged 3 commits into from Feb 28, 2023

Commits on Feb 28, 2023

  1. Remove support for aggregating reactions 

    It turns out that no clients rely on server-side aggregation of `m.annotation`
relationships: it's just not very useful as currently implemented.

It's also non-trivial to calculate.

I want to remove it from MSC2677, so to keep the implementation in line, let's
remove it here.
    @richvdh
    richvdh committed Feb 28, 2023
    Configuration menu
    Copy the full SHA
    72b2b89 View commit details
    Browse the repository at this point in the history

  2. remove dangling refs to get_aggregation_groups_for_event

    @richvdh
    richvdh committed Feb 28, 2023
    Configuration menu
    Copy the full SHA
    27c0389 View commit details
    Browse the repository at this point in the history

  3. fix up trial tests

    @richvdh
    richvdh committed Feb 28, 2023
    Configuration menu
    Copy the full SHA
    9820a57 View commit details
    Browse the repository at this point in the history