Skip to content
This repository has been archived by the owner on Jan 20, 2023. It is now read-only.

updated deprecated code #50

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ThinkDigitalSoftware
Copy link

The program was using an outdated API method and that's been updated. Also, I deleted the widget test code that was throwing an error.

@emostar
Copy link

emostar commented Aug 28, 2018

I went ahead and forked this project and added this PR (with the fix to pull gradle down) and it is available at https://github.com/emostar/audioplayer and published at https://pub.dartlang.org/packages/audioplayer2

@ThinkDigitalSoftware
Copy link
Author

ThinkDigitalSoftware commented Aug 28, 2018 via email

@emostar
Copy link

emostar commented Aug 28, 2018 via email

@ThinkDigitalSoftware
Copy link
Author

ThinkDigitalSoftware commented Aug 28, 2018 via email

@ThinkDigitalSoftware
Copy link
Author

Also, can you update the upper SDK constraints to 3.0?

@emostar
Copy link

emostar commented Aug 28, 2018

Can you file the bug report at https://github.com/emostar/audioplayer/issues, with a sample to reproduce it? I haven't looked into iOS yet with this, as I develop on Android.

@ThinkDigitalSoftware
Copy link
Author

@emostar, you don't have an issues section on your fork? I need to submit one

@emostar
Copy link

emostar commented Aug 30, 2018

@ThinkDigitalRepair It's enabled now, not sure why it was off by default.

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.

2 participants