check result is defined on fetchFinished correctly

[goldensunliu]

fix martyjs/marty#376

[taion]

Hmm - I think we'd want to do == null here instead. The intended behavior here is to treat null as a "not found" response.

Otherwise, the modules directory is actually a build artifact - you only need to change src; modules will get updated on build.

[goldensunliu]

@taion you mean !_.isNull we need to check the not condition here. also I added to the modules so our fork of marty can work for now. how do I call build?

[goldensunliu]

@taion can we merge this?

[taion]

Please change the check to a null check. I'll merge and re-release after.

[goldensunliu]

@taion all set

[taion]

Any insight to the failures on Travis?

[goldensunliu]

nothing obvious is jumping out at me hmmm

[taion]

I took a quick look at the test failure - this is an actual bug in your code. You're hitting the fetchFinished branch on https://github.com/goldensunliu/marty-lib/blob/777b066cf4b1c3482e7541d840e67b6c55e94e02/src/store/storeFetch.js#L89, but not actually marking the fetch as finished.