-
Notifications
You must be signed in to change notification settings - Fork 255
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
Set State fix and Manual Metadata Definition #95
base: master
Are you sure you want to change the base?
Conversation
radicalnerds
commented
May 5, 2017
- Fix setState call when unmounted
- Add Title metadata on Play
- Add Artwork Metadata on Play
- Default to BundleDisplayName then BundleName
- Display Name from Bundle Display Name - Title - Artwork
[songInfo setValue:albumArt forKey:MPMediaItemPropertyArtwork]; | ||
infoCenter.nowPlayingInfo = songInfo; | ||
} | ||
self.fetchedAlbumArt = true; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It seems that with your implementation you can only fetch the first song artwork, what happens if a new song from an other album is playing ?
@@ -49,10 +49,15 @@ class Player extends Component { | |||
); | |||
|
|||
ReactNativeAudioStreaming.getStatus((error, status) => { | |||
console.log('Status', status); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Log to remove
@@ -27,10 +27,10 @@ - (ReactNativeAudioStreaming *)init | |||
[self.audioPlayer setDelegate:self]; | |||
self.lastUrlString = @""; | |||
[NSTimer scheduledTimerWithTimeInterval:0.5 target:self selector:@selector(tick:) userInfo:nil repeats:YES]; | |||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Don't commit indentation changes plz
@radicalnerds thanks for your work, it looks cool and supporting album cover could be really cool 👍 |