Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Switch to latest version of audioplayers plugin which eliminates need… #195

Merged
merged 1 commit into from
Jul 7, 2021

Conversation

oxtoacart
Copy link

… for Android SDK 30

Closes getlantern/engineering#636.

@@ -47,7 +47,7 @@ dependencies:

share: ^2.0.1

audioplayers: ^0.18.3
audioplayers: ^0.19.1
Copy link
Author

Choose a reason for hiding this comment

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

Interestingly enough, the newer version of audioplayers uses an older version of androidx.core:core-ktx which doesn't require Android API level 30.

Copy link

Choose a reason for hiding this comment

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

It was a curious discovery, but is good that we only needed a newer version of audioplayers plugin

@oxtoacart oxtoacart requested a review from Crdzbird July 7, 2021 20:33
@Crdzbird
Copy link

Crdzbird commented Jul 7, 2021

LGTM

@oxtoacart
Copy link
Author

Thanks @Crdzbird !

@oxtoacart oxtoacart merged commit 96b40dd into ox/messaging Jul 7, 2021
@oxtoacart oxtoacart deleted the ox/issue186 branch July 7, 2021 22:18
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants