element-hq / element-x-android

Android Matrix messenger application using the Matrix Rust Sdk and Jetpack Compose
GNU Affero General Public License v3.0
1.08k stars 155 forks source link

Include inviter in the notification for invitation #3503

Closed bmarty closed 1 month ago

bmarty commented 1 month ago

Content

Include the sender name (or id) in the notification for invitation.

Second commit is test cleanup.

Motivation and context

More useful notification

Screenshots / GIFs

Before After
Screenshot 2024-09-20 at 10 28 00 Screenshot 2024-09-20 at 11 27 01

Note: rendering on iOS:

image

Tests

Tested devices

Checklist

github-actions[bot] commented 1 month ago

:iphone: Scan the QR code below to install the build (arm64 only) for this PR. QR code If you can't scan the QR code you can install the build via this link: https://i.diawi.com/gpfhAd

sonarcloud[bot] commented 1 month ago

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

codecov[bot] commented 1 month ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 82.67%. Comparing base (dfe9323) to head (6fc801e). Report is 32 commits behind head on develop.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## develop #3503 +/- ## ======================================== Coverage 82.67% 82.67% ======================================== Files 1731 1731 Lines 40844 40852 +8 Branches 4968 4968 ======================================== + Hits 33768 33775 +7 Misses 5315 5315 - Partials 1761 1762 +1 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.