-
Notifications
You must be signed in to change notification settings - Fork 109
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
Split VoiceMessageType
from AudioMessageType
#1664
Conversation
📱 Scan the QR code below to install the build (arm64 only) for this PR. |
Currently, for compatibility reasons, we implement MSC3245v1 which puts the voice data inside an audio message type. Though at times it seems impractical to deal with a single message type which effectively represents 2 different kinds of messages. This create a new message type called `VoiceMessageType` which is used whenever we receive an event with `"msgtype": "m.audio"` which also has the `"org.matrix.msc3245.voice": {}` field. This makes it easier to process voice messages as different entities throughout the codebase.
bc1b061
to
db717e1
Compare
Kudos, SonarCloud Quality Gate passed! |
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## develop #1664 +/- ##
===========================================
- Coverage 63.22% 63.21% -0.02%
===========================================
Files 1228 1228
Lines 31693 31706 +13
Branches 6526 6531 +5
===========================================
+ Hits 20039 20042 +3
- Misses 8656 8664 +8
- Partials 2998 3000 +2
☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks!
Currently, for compatibility reasons, we implement MSC3245v1 which puts the voice data inside an audio message type. Though at times it seems impractical to deal with a single message type which effectively represents 2 different kinds of messages.
This PR creates a new message type called
VoiceMessageType
which is used whenever we receive an event with"msgtype": "m.audio"
which also has the"org.matrix.msc3245.voice": {}
field. This makes it easier to process voice messages as different entities throughout the rest of the codebase.