Skip to content
This repository has been archived by the owner on Mar 16, 2021. It is now read-only.

RxJava2 support #54

Merged
merged 7 commits into from
Jan 19, 2017
Merged

RxJava2 support #54

merged 7 commits into from
Jan 19, 2017

Conversation

StefMa
Copy link
Contributor

@StefMa StefMa commented Jan 6, 2017

RxJava2 support

This is a first draft of supporting RxJava2.
Included:
The fully helper class RxTiPresenterDisposableHandler.
A part of RxTiPresenterUtils. Currently only isViewReady().

I've created a new rx2 module which can be plugged in like the RxJava (1) module (need to be uploaded to maven of course)...

TODOs

  • Add deliverLatestCacheToView()
  • Add deliverLatestToView()
  • Add deliverToView()

@StefMa StefMa added the rx label Jan 6, 2017
This was referenced Jan 6, 2017
mDisposableHandler.manageViewDisposable(testObserver);
mPresenter.detachView();

// Think about that.
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fix this like in the rxmodule. See #58 and #55

@passsy
Copy link
Contributor

passsy commented Jan 19, 2017

Let's add the deliver*ToView() in a separate PR

@StefMa
Copy link
Contributor Author

StefMa commented Jan 19, 2017 via email

@passsy passsy merged commit c091ac5 into GCX-HCI:master Jan 19, 2017
@passsy passsy deleted the feature/rx2 branch January 19, 2017 17:09
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Development

Successfully merging this pull request may close these issues.

2 participants