Skip to content
This repository has been archived by the owner on Apr 12, 2022. It is now read-only.

Add internal recording of voice messages #3314

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from

Conversation

awesome-manuel
Copy link
Contributor

Pull Request Checklist

  • Changes has been tested on an Android device or Android emulator with API 16
  • UI change has been tested on both light and dark themes
  • Pull request is based on the develop branch
  • Pull request updates CHANGES.rst
  • Pull request includes screenshots or videos if containing UI changes
  • Pull request includes a sign off

Signed-off-by: Manuel Stahl <[email protected]>
Change-Id: If753ce84a66d5c1ac5d176267dccb16c147cf6de
@DoM1niC
Copy link

DoM1niC commented Oct 9, 2019

I recommend to add 4 improving quality of Records...

mRecorder.setAudioChannels(2);
mRecorder.setAudioSamplingRate(44100);
mRecorder.setAudioEncodingBitRate(64000);

I merge this on my Fork without any Problems :) THX anyway

@DoM1niC
Copy link

DoM1niC commented Oct 9, 2019

@awesome-manuel Could you add Ear hearing Mode like simulated Phone Call ?

@awesome-manuel
Copy link
Contributor Author

@awesome-manuel Could you add Ear hearing Mode like simulated Phone Call ?

For sure not in this PR, could be added later. We don't need it, so probably someone else has to contribute that code.

@DoM1niC
Copy link

DoM1niC commented Oct 15, 2019

@awesome-manuel Could you add Ear hearing Mode like simulated Phone Call ?

For sure not in this PR, could be added later. We don't need it, so probably someone else has to contribute that code.

I use a own Fork of this & want's to make it more friendly 4 my Community like alternative to WhatsApp and so on....

@tuxayo
Copy link
Contributor

tuxayo commented Mar 18, 2020

Is there anything to take here for RiotX? At least the design principles?

@tuxayo
Copy link
Contributor

tuxayo commented Mar 18, 2020

RiotX issue: element-hq/element-android#29

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants