Skip to content
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

Bug Fix - IllegalStateException in stopAudioRecording + Polishes #3453

Merged
merged 3 commits into from
Nov 29, 2023

Conversation

rapterjet2004
Copy link
Contributor

@rapterjet2004 rapterjet2004 commented Nov 20, 2023

I decided to also implement some additional Bug fixes and some refactoring, in addition to the IllegalStateException bug fix. Figured it would be better than multiple PR's.

  • Fixes IllegalStateException
  • Refactored code to be clearer and manage the state of the MediaRecorder object directly
  • Saves state of voice recording upon life cycle changes like what telegram has
  • Waveform is now saved to storage, so it isn't recreated every time the user exits the activity
  • Clicking the expanded restart record button while preview is playing stops the preview
  • File captions don't crash now, when the itemView is clicked
  • Note to Self now allows messages expired or all messages to be deleted

🏁 Checklist

  • ⛑️ Tests (unit and/or integration) are included or not needed
  • 🔖 Capability is checked or not needed
  • 🔙 Backport requests are created or not needed: /backport to stable-xx.x
  • 📅 Milestone is set
  • 🌸 PR title is meaningful (if it should be in the changelog: is it meaningful to users?)

@rapterjet2004 rapterjet2004 added bug Something isn't working 2. developing Work in progress feature: 🗨️ chat labels Nov 20, 2023
@rapterjet2004 rapterjet2004 self-assigned this Nov 20, 2023
@rapterjet2004 rapterjet2004 added 3. to review Waiting for reviews and removed 2. developing Work in progress labels Nov 21, 2023
@rapterjet2004 rapterjet2004 marked this pull request as ready for review November 21, 2023 16:10
@rapterjet2004 rapterjet2004 marked this pull request as draft November 22, 2023 13:58
@rapterjet2004 rapterjet2004 changed the title Bug Fix - IllegalStateException in stopAudioRecording Bug Fix - IllegalStateException in stopAudioRecording + Polishes Nov 22, 2023
@rapterjet2004 rapterjet2004 added 2. developing Work in progress and removed 3. to review Waiting for reviews labels Nov 22, 2023
@rapterjet2004 rapterjet2004 force-pushed the issue-3302-bug-fix branch 2 times, most recently from 5847258 to 500ba2c Compare November 24, 2023 20:42
@rapterjet2004 rapterjet2004 added 3. to review Waiting for reviews and removed 2. developing Work in progress labels Nov 24, 2023
@rapterjet2004 rapterjet2004 marked this pull request as ready for review November 24, 2023 20:52
@rapterjet2004 rapterjet2004 added 2. developing Work in progress and removed 3. to review Waiting for reviews labels Nov 25, 2023
@rapterjet2004 rapterjet2004 added 3. to review Waiting for reviews and removed 2. developing Work in progress labels Nov 27, 2023
@mahibi
Copy link
Collaborator

mahibi commented Nov 28, 2023

i get "mediarecorder went away with unhandled events" when doing:

  1. start and lock media recording
  2. click delete

@mahibi
Copy link
Collaborator

mahibi commented Nov 28, 2023

great improvements 👍
from my pov this can be merged when the minor comments are solved.

@mahibi
Copy link
Collaborator

mahibi commented Nov 28, 2023

In general it's better to seperate the changes into multiple PR's, especially when different features are touched.

@mahibi mahibi added this to the 18.1.0 milestone Nov 28, 2023
@mahibi
Copy link
Collaborator

mahibi commented Nov 28, 2023

/backport to stable-18.0

Copy link
Contributor

Codacy

Lint

TypemasterPR
Warnings8585
Errors00

SpotBugs

CategoryBaseNew
Bad practice66
Correctness1010
Dodgy code123123
Internationalization55
Malicious code vulnerability33
Performance88
Security22
Total157157

@rapterjet2004 rapterjet2004 requested a review from mahibi November 28, 2023 13:49
- Implement MediaRecorderState handling
- Fixed lifecycle bug - recording and locked UI now ends after app exits
- Saves Waveform to storage after initial loading
- Fixes File caption crashes
- A couple other bugs

Signed-off-by: Julius Linus <[email protected]>
Signed-off-by: rapterjet2004 <[email protected]>
@mahibi mahibi force-pushed the issue-3302-bug-fix branch from 07670ff to 2adad16 Compare November 29, 2023 10:38
@mahibi mahibi merged commit 5f53005 into master Nov 29, 2023
12 of 16 checks passed
@delete-merged-branch delete-merged-branch bot deleted the issue-3302-bug-fix branch November 29, 2023 10:39
Copy link
Contributor

APK file: https://www.kaminsky.me/nc-dev/android-artifacts/3453-talk.apk

qrcode

To test this change/fix you can simply download above APK file and install and test it in parallel to your existing Nextcloud Talk app.

Copy link
Contributor

Hello there,
Thank you so much for taking the time and effort to create a pull request to our Nextcloud project.

We hope that the review process is going smooth and is helpful for you. We want to ensure your pull request is reviewed to your satisfaction. If you have a moment, our community management team would very much appreciate your feedback on your experience with this PR review process.

Your feedback is valuable to us as we continuously strive to improve our community developer experience. Please take a moment to complete our short survey by clicking on the following link: https://cloud.nextcloud.com/apps/forms/s/i9Ago4EQRZ7TWxjfmeEpPkf6

Thank you for contributing to Nextcloud and we hope to hear from you soon!

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

Successfully merging this pull request may close these issues.

IllegalStateException in stopAudioRecording
3 participants