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

feat: [Example App] keep recording on reboot + allow to delete them #122

Conversation

dprevost-LMI
Copy link
Contributor

@dprevost-LMI dprevost-LMI commented Oct 20, 2024

  • Add getRecordedAudios that fetches m4a audio files from the cache for Android or document for iOS.
  • Add a delete button to delete all the recordings, but an app reboot is required to see the effect!

Copy link
Contributor

@kuldip-simform kuldip-simform left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@dprevost-LMI I have added some comments. Can you please resolve them so we can approve this PR and merge this.

I like this very much. This improves user experience who are using our example app to try out this library.

Thank you so much for your efforts to improve this library.

Copy link
Contributor

@kuldip-simform kuldip-simform left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for quick update.
Just 2 small comments.

kuldip-simform
kuldip-simform previously approved these changes Oct 23, 2024
@mukesh-simform mukesh-simform changed the base branch from master to develop November 8, 2024 11:27
@mukesh-simform mukesh-simform dismissed kuldip-simform’s stale review November 8, 2024 11:27

The base branch was changed.

@dprevost-LMI dprevost-LMI force-pushed the show-recoring-on-boot-and-delete-recording branch from a0510c0 to c164249 Compare November 9, 2024 00:43
@dprevost-LMI
Copy link
Contributor Author

dprevost-LMI commented Nov 9, 2024

Hey, I rebased and revalidated that we can see the icon on iOS!
Screenshot 2024-11-08 at 7 58 38 PM

@kuldip-simform do you mind reapproving, 🙏 , I rebase from develop to fix a conflict

@mukesh-simform mukesh-simform merged commit 9bff4cd into SimformSolutionsPvtLtd:develop Nov 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FEATURE] [Example App] Keep recording on reboot + ability to delete them all
3 participants