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

Implement filtering #587

Open
babolivier opened this issue Nov 7, 2018 · 8 comments
Open

Implement filtering #587

babolivier opened this issue Nov 7, 2018 · 8 comments
Labels
are-we-synapse-yet This issue or PR involves Sytests in AWSY C-Sync-API good first issue Want to help with Dendrite? These are the issues to start with! T-Enhancement New features, changes in functionality, improvements in performance, or user-facing enhancements.

Comments

@babolivier
Copy link
Contributor

This issue is tracking the progress of the implementation of filtering on the CS API. AFAIK @CromFr has already been working on implementing that on /sync. Will also need to be implemented on /messages when that endpoint itself gets implemented.

@CromFr
Copy link
Contributor

CromFr commented Nov 7, 2018

Yay filters ! 🎉

My current filter-related PRs:

I should have in my local repo some branches for other event types, and the final IncrementalSync/CompleteSync implementation, but requires some additional work to be sane & clean.

@babolivier
Copy link
Contributor Author

Thanks for the info!
For the record, and for maximum clarity, this issue being created doesn't mean that filtering will be addressed immediately (it basically means that I needed an open issue to make a TODO comment that's referring to it in a PR I'm currently working on).
Hopefully there'll soon be enough bandwidth to deal with it, though.

@kegsay
Copy link
Member

kegsay commented Aug 24, 2020

Sytests:

    × Can request federation format via the filter
    × Check creating invalid filters returns 4xx

@kegsay kegsay added are-we-synapse-yet This issue or PR involves Sytests in AWSY good first issue Want to help with Dendrite? These are the issues to start with! labels Aug 24, 2020
@kegsay
Copy link
Member

kegsay commented Feb 8, 2021

@neilalexander did you do this in #1721 ?

@neilalexander
Copy link
Contributor

We have filtering in /sync but I don't think we have it in /messages yet.

@spaetz
Copy link
Contributor

spaetz commented Apr 24, 2022

So, was that implemented by #2347 and can be closed then?

@kegsay
Copy link
Member

kegsay commented May 23, 2022

There are still 2 outstanding tests which are failing.

@kegsay
Copy link
Member

kegsay commented Dec 5, 2022

Still failing.

@kegsay kegsay added the T-Enhancement New features, changes in functionality, improvements in performance, or user-facing enhancements. label Dec 5, 2022
@devonh devonh removed this from the Dendrite 0.5.0 milestone Jan 23, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
are-we-synapse-yet This issue or PR involves Sytests in AWSY C-Sync-API good first issue Want to help with Dendrite? These are the issues to start with! T-Enhancement New features, changes in functionality, improvements in performance, or user-facing enhancements.
Projects
None yet
Development

No branches or pull requests

7 participants