Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implement MSC3966: a push rule condition to check if an array contains a value #3180

Merged
merged 4 commits into from
Mar 7, 2023

Conversation

clokep
Copy link
Member

@clokep clokep commented Feb 28, 2023

Implements the stable version of MSC3966 which adds a push rule condition to check for a value in an array.

MSC3966 is in the final comment period, but not yet merged. We must wait until it finishes FCP before merging.

This is currently based on #3179, I'll rebase once that is merged.

Checklist

  • Tests written for new code (and old code if feasible)
  • Linter and other CI checks pass
  • Sign-off given on the changes (see CONTRIBUTING.md)

Here's what your changelog entry will look like:

✨ Features

  • Implement MSC3966: a push rule condition to check if an array contains a value (#3180).

@clokep clokep added the T-Task Tasks for the team like planning label Feb 28, 2023
@clokep clokep changed the title Implement MSC3966 to match values in an array in push rule conditions. Implement MSC3966: a push rule condition to check if an array contains a value Mar 1, 2023
@clokep clokep added T-Enhancement and removed T-Task Tasks for the team like planning labels Mar 1, 2023
@clokep clokep removed the request for review from alunturner March 6, 2023 16:18
@clokep
Copy link
Member Author

clokep commented Mar 7, 2023

MSC3966 has passed FCP, so this can now be merged. 🎉

@clokep
Copy link
Member Author

clokep commented Mar 7, 2023

Had to tweak the tests for the linter!

@richvdh richvdh added this pull request to the merge queue Mar 7, 2023
Merged via the queue into matrix-org:develop with commit bcf3bba Mar 7, 2023
@clokep clokep deleted the event-contains branch March 7, 2023 16:59
su-ex added a commit to SchildiChat/matrix-js-sdk that referenced this pull request Apr 21, 2023
* Allow via_servers property in findPredecessor (update to MSC3946) ([\matrix-org#3240](matrix-org#3240)). Contributed by @andybalaam.
* Fire `closed` event when IndexedDB closes unexpectedly ([\matrix-org#3218](matrix-org#3218)).
* Implement MSC3952: intentional mentions ([\matrix-org#3092](matrix-org#3092)). Fixes element-hq/element-web#24376.
* Send one time key count and unused fallback keys for rust-crypto ([\matrix-org#3215](matrix-org#3215)). Fixes element-hq/element-web#24795. Contributed by @florianduros.
* Improve `processBeaconEvents` hotpath ([\matrix-org#3200](matrix-org#3200)).
* Implement MSC3966: a push rule condition to check if an array contains a value ([\matrix-org#3180](matrix-org#3180)).
* indexddb-local-backend - return the current sync to database promise … ([\matrix-org#3222](matrix-org#3222)). Contributed by @texuf.
* Revert "Add the call object to Call events" ([\matrix-org#3236](matrix-org#3236)).
* Handle group call redaction ([\matrix-org#3231](matrix-org#3231)). Fixes vector-im/voip-internal#128.
* Stop doing O(n^2) work to find event's home (`eventShouldLiveIn`) ([\matrix-org#3227](matrix-org#3227)). Contributed by @jryans.
* Fix bug where video would not unmute if it started muted ([\matrix-org#3213](matrix-org#3213)). Fixes element-hq/element-call#925.
* Fixes to event encryption in the Rust Crypto implementation ([\matrix-org#3202](matrix-org#3202)).
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