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

update return type of result when user stop the audio player #283

Merged
merged 1 commit into from
Nov 26, 2024

Conversation

himanshuGandhiSimform
Copy link
Collaborator

Update return type of data from native to flutter

final audioInfo =
await _methodChannel.invokeMethod(Constants.stopRecording);
return List.from(audioInfo ?? []);
Future<Map<String,dynamic>> stop() async {
Copy link
Collaborator

Choose a reason for hiding this comment

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

File looks unformatted

@ujas-m-simformsolutions
Copy link
Collaborator

Also please bump the version and update changelog mentioning minor internal restructure and mention breaking change of previous PR.

@himanshuGandhiSimform himanshuGandhiSimform force-pushed the update_native_data_return_type branch from 1c751ec to 5aff7e3 Compare March 15, 2024 10:58
hemantbeast added a commit to hemantbeast/audio_waveforms that referenced this pull request May 28, 2024
@himanshuGandhiSimform himanshuGandhiSimform force-pushed the update_native_data_return_type branch 2 times, most recently from d752903 to 06e27ca Compare November 26, 2024 10:19
lib/src/base/audio_waveforms_interface.dart Show resolved Hide resolved
Comment on lines 317 to 318
if (audioInfo[Constants.resultDuration] != null) {
var duration = audioInfo[Constants.resultDuration];
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
if (audioInfo[Constants.resultDuration] != null) {
var duration = audioInfo[Constants.resultDuration];
final duration = audioInfo[Constants.resultDuration];
if(duration != null) {
...
}

@himanshuGandhiSimform himanshuGandhiSimform force-pushed the update_native_data_return_type branch from 06e27ca to df3a36f Compare November 26, 2024 12:00
@ujas-m-simformsolutions ujas-m-simformsolutions merged commit 3f58d41 into main Nov 26, 2024
@ujas-m-simformsolutions ujas-m-simformsolutions deleted the update_native_data_return_type branch November 26, 2024 12:20
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.

2 participants