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

MSC3517: "Mention" Pushrule #3517

Closed
Closed
Changes from 3 commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
51 changes: 51 additions & 0 deletions proposals/3517-mention-pushrule.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,51 @@
# MSC3517: Mention Pushrule

Pings in matrix can be inconsistent for someone coming from an environment where pings are explicit
(e.g. Discord, Telegram, Slack, Whatsapp, etc.)

Currently, personal pings are governed by 2 push rules; match on display name, and match on username.

However, due to a variety of reasons, these push rules can have false-positives, and a suitable
alternative that only gives notifications on explicit pings does not exist.

## Proposal

This proposal aims to change that, adding the following default push rule:

```json
{
"rule_id": ".m.rule.pings_mxid",
"default": true,
"enabled": true,
"conditions": [
{
"kind": "mxid_ping"
}
],
"actions": [
"notify",
{
"set_tweak": "sound",
"value": "default"
},
{
"set_tweak": "highlight"
}
]
}
```

And the following condition; `mxid_ping`.

Currently, this condition should trigger if the user's MXID is found in `content.body`,
or `content.formatted_body`.
ShadowJonathan marked this conversation as resolved.
Show resolved Hide resolved

Rationale:
> This is called MXID "ping", as "mention" and "contains" would not be good fits for
> future-proofing, one of the benefits of a separate condition is that, in the future,
> once more comprehensive mention techniques come to fruition (such as putting mentions
> in an array), the rule could still automatically apply with a small tweak.

## Unstable prefix

The unstable should be `.nl.automatia.rule.pings_mxid`