Implement Intentional Mentions (MSC3952) on Mobile

[daniellekirkwood]

Your use case

MSC3952 is nearly out of FCP. Once completed we will need to implement and stabilise it.

• On web: stabilise
• On mobile EX:
  • [Story] Mention a user in message #2066
  • [Story] Show that I have been mentioned in the push notification #2065
  • [Story] Show a user has been mentioned in the timeline #2062

For cross-linking (originally from #886 (comment)):

• Support MSC3952 intentional mentions element-android#8489
• Support MSC3952 intentional mentions element-ios#7567
• push: Add support for intentional mentions push rules push: Add support for intentional mentions push rules  ruma/ruma#1550 / Implement Matrix v1.7 changes ruma/ruma#1520 -- needed for Element X, I think?
• There's also Implement MSC3952: Intentional mentions matrix-org/matrix-rust-sdk#2606

[daniellekirkwood]

@giomfo Created this 'holding' issue based on our conversation today (before I forget). I will add more to it tomorrow

[clokep]

@daniellekirkwood I filed matrix-org/synapse#15487 for the Synapse side.

[daniellekirkwood]

This doesn't need to be on the WAT board anymore -- @Johennes & @clokep did we get the Synapse and EleWeb side of this sorted? Should I move it to a mobile only issue and assign to the right folks?

[Johennes]

There is nothing left to do on the Element Web side AFAIK.

[clokep]

did we get the Synapse and EleWeb side of this sorted?

Synapse supports it, as does Element Web & its components. It would be nice to finish off support for Matrix 1.7 in Synapse so clients are able to easily see this is supported, but it is not a requirement since clients can see the push rule and key off that.

Should I move it to a mobile only issue and assign to the right folks?

For cross-linking (originally from #886 (comment)):

• Support MSC3952 intentional mentions element-android#8489
• Support MSC3952 intentional mentions element-ios#7567
• push: Add support for intentional mentions push rules  ruma/ruma#1550 / Implement Matrix v1.7 changes ruma/ruma#1520 -- needed for Element X, I think? There's also Implement MSC3952: Intentional mentions matrix-org/matrix-rust-sdk#2606

[daniellekirkwood]

@manuroe @giomfo @VolkerJunginger

We've done most of what we need on the server and web side here so I've updated the task to be an EX one. Let us know if you have questions, otherwise we'll leave this to prioritise on your roadmap. Thanks.

[giomfo]

@manuroe @langleyd FYI the intentional mentions (MSC3952) has been taken into account when we "Expose the push notification settings #1959" in Rust SDK. Some updates have been applied on this implementation to improve it like this PR.
I consider EX is ready to support MSC3952 - You may double check with EX Feature team
About Legacy Element mobile clients, nothing has been done, the mention handling would depend only on the legacy push rules. I think the new push rules are ignored by default on these clients.

[manuroe]

The platform team is on it, assembling the different pieces made by other teams. Thanks all!

I edited the issue with user stories that cover intentional mentions on EX mobile. They are part of the broader [Epic] Mentions that will make mentions work for real even in E2EE rooms!