search

RxSwiftCommunity / RxRealm

RxSwift extension for RealmSwift's types
MIT License
1.15k stars 265 forks source link

Object property notifications #67

Closed ushu closed 7 years ago

ushu commented 7 years ago

Add a new Observable.changes(from:) to observe property changes.

icanzilb commented 7 years ago

thanks! I'm gonna have a look at this probably today on the train 👍🏼

icanzilb commented 7 years ago

@ushu code looks good and I like it that you split the property changes into separate next events - that makes a lot of sense. The only thing I'd change is the name because we will have Observable.from(object: Object) and Observable.changes(from: Object). I think that if we change the latter to Observable.propertyChanges(from: Object) would be more obvious for developers which one does what. What do you think about that suggestion?