Skip to content

Upstream changes#2

Closed
jeffmon wants to merge 19 commits into
agathongroup:agathonfrom
doublesymmetry:dev
Closed

Upstream changes#2
jeffmon wants to merge 19 commits into
agathongroup:agathonfrom
doublesymmetry:dev

Conversation

@jeffmon

@jeffmon jeffmon commented Jan 11, 2020

Copy link
Copy Markdown
Member

No description provided.

jonnyberanek and others added 15 commits October 20, 2019 19:51
Under "Events", the export to view all event types was mislabeled as `TrackPlayerEventTypes`

Changed documentation to match correct name of `TrackPlayerEvents`. See: https://github.com/react-native-kit/react-native-track-player/blob/97692c6894c5ffaae9ad5772c4db0e24c420ac0d/lib/index.js#L202-L203
…xception: invalid channel for service notification: null (#723)
* Call `playback-track-changed` for penultimate track too

* Fix duplicate `playback-track-changed` call

* Fixed code style, reduced condition checking
…control specific timing of queue events. Still advances to the next track without further JS code. (#822)
(cherry picked from commit 99f8524)
(cherry picked from commit 170c14a)
@jeffmon

jeffmon commented Jan 11, 2020

Copy link
Copy Markdown
Member Author

@lukebaker It looks like I don't have write access to this repo, any chance you can take a look at this? Thanks!

@lukebaker

Copy link
Copy Markdown

@jeffmon I've given you write access. Do you still want me to take a look at this?

@jeffmon

jeffmon commented Jan 13, 2020

Copy link
Copy Markdown
Member Author

@lukebaker Thanks for that! Would it be wise to merge in all of the things from their dev branch in the first place? And yes, would you be able to take a look? 😬

@lukebaker lukebaker mentioned this pull request Jan 14, 2020
@lukebaker

Copy link
Copy Markdown

Seem my changes in #4. I think that does what we want.

Guichaguri and others added 4 commits January 19, 2020 12:57
* fix: Remove iOS SwiftAudio unnecessary files

* Remove extra line

* Update .npmignore

* Podspect exclude files
@lukebaker lukebaker closed this Apr 16, 2020
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.

10 participants