Skip to content

Only send voice messages as GroupMe voice notes - #33

Merged
Fizzadar merged 1 commit into
mainfrom
iFixRobots/voice-only-audio
Oct 27, 2026
Merged

Fizzadar merged 1 commit into
mainfrom
iFixRobots/voice-only-audio

Conversation

@iFixRobots

Copy link
Copy Markdown
Contributor

Since #31, every m.audio event goes out as a GroupMe voice note, not just voice messages. A plain audio file, like an .mp3 with a caption, arrives in GroupMe as a voice note and silently loses its caption:

case event.MsgAudio:
	text = ""

The room capabilities only advertise voice messages (org.matrix.msc3245.voice, audio/mp4). GroupMe has nothing for plain audio files: its web client uploads them as group files.

Audio without the voice flag now fails as unsupported again, which is what happened before #31. Voice messages are unchanged.

Before After
Voice message from Beeper GroupMe voice note GroupMe voice note
Audio file with caption Voice note, caption dropped Unsupported notice

Changelog: Internal, no change for voice messages.

Before / after

I haven't run this on a device yet; it needs a staging bridge build.

View in Indent
Tag @indent to continue the conversation here.

@indent indent Bot added the to be tested label Oct 27, 2026
@indent

indent Bot commented Oct 27, 2026 •

Copy link
Copy Markdown
Contributor

Issues

No issues found.

CI Checks

All CI checks passed on 83dee26.

Review agents

Select any unchecked box below to run or rerun that agent.

Passed (1)
  • Indent Review Agent · No issues; the voice-only check is correct.
Full results

Indent Review Agent

  • Summary: No issues; the voice-only check is correct.
  • Last ran on commit: 83dee266
  • Latest result
    {
      "summary": "No issues; the voice-only check is correct.",
      "findings": []
    }

@Fizzadar
Fizzadar merged commit 7fbca16 into main Oct 27, 2026
5 checks passed
@Fizzadar
Fizzadar deleted the iFixRobots/voice-only-audio branch October 27, 2026 18:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

2 participants