airbnb / mavericks

Mavericks: Android on Autopilot
https://airbnb.io/mavericks/
Apache License 2.0
5.83k stars 500 forks source link

Outdated documentation #237

Closed mradzinski closed 5 years ago

mradzinski commented 5 years ago

Hi guys! I think this: https://github.com/airbnb/MvRx/wiki/Advanced-Concepts#dependent-requests is kind of outdated. It makes reference to onSubscribe, which I'm not entirely sure if its an internal Airbnb method or an old method which got replaced by subscribe and asyncSubscribe.

Let me know, I can PR changes if needed

gpeal commented 5 years ago

Thank you, I have updated it to:

init {
    asyncSubscribe(MyState::request1), onSuccess = {
        fetchRequest2()
    })
}