search

tajchert / BusWear

EventBus for Android Wear devices.
Apache License 2.0
257 stars 23 forks source link

Update BusWear with new version of EventBus #16

Closed btkelly closed 7 years ago

btkelly commented 8 years ago

EventBus 3.0 has been released, update BusWear to use the new version.

With this update, investigate if EventBus can now be used a dependency vs a full inclusion of the code. This will allow quicker updates as new versions of EventBus are released.

btkelly commented 8 years ago

I've started looking into this and it seems like EventBus is still locked down which makes it hard to include it as a dependency. I'm still playing around but it looks like it's still required to include EventBus in the project like it is currently.

btkelly commented 8 years ago

@greenrobot Would it be in the pipeline to allow for easier extension of the EventBus library? Right now many methods and fields are private which makes it difficult to add additional features.

greenrobot commented 8 years ago

Can you be more specific about needed changes? It's not enough, just to register for all events (Object) and forward them?

btkelly commented 8 years ago

Thanks for responding @greenrobot but I think I was able to accomplish using the dependency only with the current EventBus library in PR #18. Took a bit of refactoring of the BusWear API but should provide the same functionality.