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

Proposition: new Connection API (replace RxBleConnection) #239

Closed
wants to merge 8 commits into from

Conversation

dariuszseweryn
Copy link
Owner

It aims to provide a more concise API that utilises newer RxJava 1 API like Single and Completable as well as a clean extension point for future functionality “knobs”.

This Pull Request is made to ignite a discussion about the future API.

It aims to provide a more concise API that utilises newer `RxJava 1` API like `Single` and `Completable` as well as a clean extension point for future functionality “knobs”.
…, `Completable`) with possibility of managed radio behaviour.
dariuszseweryn added a commit that referenced this pull request Aug 3, 2017
…lishConnection(SetupOptions)`

(#240 #224) This new function is a perfect extension point to introduce a new public API #239 which will return the new API without changing the Major version number. Till then all new public classes were moved to the `internal` package and should not be publicly available.
dariuszseweryn added a commit that referenced this pull request Aug 3, 2017
…nal package.

(#240 #224) Will become public when new API #239 will be ready.
…nection discoverServices() return type. Removed specialized discoverService(DiscoverySetup).
@RobLewis
Copy link

This was mentioned as being part of the 1.5.0 release. Is that still the plan?

@uKL
Copy link
Collaborator

uKL commented Jan 11, 2018

I don't think so, I think we'll postpone it. It looks like there will be 1.5.0, then RxJava2, then new API.

Am I right @dariuszseweryn ?

@dariuszseweryn
Copy link
Owner Author

This will not be included in 1.5.0. We currently have limited capacity and decided that support for RxJava 2 is of greater importance. The new API will get included once it is ready... Hard to say when.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants