Skip to content

Conversation

@reivilibre
Copy link
Contributor

Follows: #18756

Implements: MSC4306

Base: rei/t2_msc4306_conflict

This pull request is commit-by-commit review friendly.

  1. Add MSC4306's default push rules

  2. Add batch get_subscribers_to_thread

  3. Optimise push rule evaluation by getting subscribers in batch

@reivilibre reivilibre changed the title Implement the push rules for experimental [MSC4306: Thread Subscriptions](https://github.com/matrix-org/matrix-doc/issues/4306). Implement the push rules for experimental MSC4306: Thread Subscriptions. Aug 1, 2025
@reivilibre reivilibre marked this pull request as ready for review August 1, 2025 12:42
@reivilibre reivilibre requested a review from a team as a code owner August 1, 2025 12:42
Copy link
Contributor

@MadLittleMods MadLittleMods left a comment

Choose a reason for hiding this comment

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

I don't know what I'm looking at when at when it comes to push rules

@MadLittleMods MadLittleMods requested a review from a team August 4, 2025 22:33
Copy link
Member

@anoadragon453 anoadragon453 left a comment

Choose a reason for hiding this comment

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

No hard blockers from me, but it would be good to fix/add comments for the rest of the threads on this PR.

# specify this in the `bdist_wheel` config below.
py_limited_api=True,
# We always build in release mode, as we can't distinguish
# We always build in release mode, as we can't distinguish
Copy link
Member

Choose a reason for hiding this comment

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

FYI this has already been fixed in 25289b6.

But no harm in including it in this PR.

@reivilibre reivilibre force-pushed the rei/t2_msc4306_conflict branch from aed6d75 to 29c1ae4 Compare August 5, 2025 16:19
Base automatically changed from rei/t2_msc4306_conflict to develop August 5, 2025 18:22
@reivilibre reivilibre enabled auto-merge (squash) August 5, 2025 18:59
@reivilibre
Copy link
Contributor Author

CI failures are not related to this PR (ongoing issue on develop)

@reivilibre reivilibre disabled auto-merge August 6, 2025 14:33
@reivilibre reivilibre merged commit 6514381 into develop Aug 6, 2025
241 of 273 checks passed
@reivilibre reivilibre deleted the rei/t2_pushrules branch August 6, 2025 14:33
reivilibre added a commit that referenced this pull request Sep 9, 2025
…ollow newer draft. (#18846)

Follows: #18762

Implements: MSC4306

Closes: #18431
Closes: #18437

Move the MSC4306 push rules to a new kind `postcontent` 

Prevent users from creating user-defined `postcontent` rules 

---------

Signed-off-by: Olivier 'reivilibre <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants