Peter Livesey
Peter Livesey
I think this is no longer true? Looks like the frameworks are actually built with i386? May be wrong though... ``` Another note: Included frameworks only contain arm64 and armv7...
Btw, ran tests on another project and found that it looks like there is now iOS 10 regression (phew). Going to leave this issue open since I was unable to...
The reason that it's on the main thread is because it's updated on the main thread. So, the model modelUpdatesListeners is read on the main thread, so should be written...
Yeah, looking back through the docs, I found: https://plivesey.github.io/RocketData/pages/110_threading.html I think there's significant work to make data providers thread safe. Also, I'd want to make sure that code didn't affect...
I think the fact that `CollectionDataProvider` is chance and is not fully correct. There are several points in the code which assumes the main thread and you're introducing several race...
Also `mergeModel(_)` -> `merge(_)`
It sounds like it's because your function probably looks like: ``` send(_ request: Requestable, completion: @escaping (Result)->Void) -> NetworkTask ``` But in your code here, you're asking for [T] instead...
Hmmmm...yeah, I can reproduce and you're right. It looks like lazy loading these values maybe wasn't the best choice. Really, there shouldn't be any getter in this method, which isn't...
Oh man...thanks for reporting this! I didn't notice this, but clearly causing problems. I'll add it to my pr #10.
I realize there's too much in this PR for one commit, but it seems like this library isn't really being maintained anymore. Happy to split this up when you get...