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

Added event status STREAMING #8

Merged
merged 1 commit into from
Aug 3, 2016
Merged

Added event status STREAMING #8

merged 1 commit into from
Aug 3, 2016

Conversation

brettpappas
Copy link
Collaborator

New STREAMING event which passes back current progress while audio is playing or paused.

New STREAMING event which passes back current progress while audio is playing or paused.
@tlenclos
Copy link
Owner

tlenclos commented Aug 3, 2016

How is it behaving with audio stream without duration like the one in the example app ?

@brettpappas
Copy link
Collaborator Author

I had only tested with fixed length mp3 files. I will try to test with the live stream example today.

@brettpappas
Copy link
Collaborator Author

Normally, the status for STREAMING will have duration and progress where duration is a fixed float number representing the length of the audio (in seconds) and the progress is a float that increases as you play the audio.

With a live stream, the duration stays at 0 and the progress increases from 0 to however long you listen.

As for the seekToTime, goBack and goForward functions - they are simply ignored when using a live stream. We could add code in the example where IF the duration is zero, then hide (or disable) the goBack and goForward buttons).

See the screenshot below for a live stream from the console:

screen shot 2016-08-03 at 8 23 16 am

@tlenclos
Copy link
Owner

tlenclos commented Aug 3, 2016

👍 , thanks !

@tlenclos tlenclos merged commit 74b1a30 into tlenclos:master Aug 3, 2016
@varun-raj
Copy link

How can I use this event? Is there any documentation for it?

@tlenclos
Copy link
Owner

tlenclos commented Jan 5, 2017

It is not documented yet because it's only available on iOS.

@varun-raj
Copy link

Can you give me some reference? I need this feature very desperately. :)

@tlenclos
Copy link
Owner

tlenclos commented Jan 7, 2017

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.

3 participants