Implement clarifications for the PUT _matrix/client/v3/pushrules/{scope}/{kind}/{ruleID}
endpoint
#2870
Labels
C-User-API
F-Push
spec-compliance
Fix something that doesn't comply with the specs
T-Task
Refactoring, removal, replacement, enabling or disabling functionality, other engineering tasks.
Two PRs were merged lately with clarifications to push rules, and specifically the
PUT _matrix/client/v3/pushrules/{scope}/{kind}/{ruleID}
endpoint:PUT /pushrules/{scope}/{kind}/{ruleId}
matrix-spec#1319.m.rule.master
has a higher priority than any push rule matrix-spec#1320Although those were partly based on Dendrite's behavior, a few changes are needed in
PutPushRuleByRuleID
to be spec-compliant:m.rule.master
is always the rule with the highest priority, even higher than user-defined ones.I believe a database migration needs to be written to make sure that no server-default pushrules were overriden and
m.rule.master
is the first.The text was updated successfully, but these errors were encountered: