With these changes we always do 2 request per request (cache + API) when the key is not saved (it happens because we don't have unlimited space). That's the reason of getting all the cache keys before.
Of course, we can live with that, but it goes against performance directly.

Ah that was not the goal! The idea was that for each fetch you attempt to get that cache data (using the request as the key). If there is cache then it simply returns and you have the data, if not it gets the data and caches it. I wanted to try it this way to test performance as it might be a couple more fetches but they are very very small fetches only in the case that there isnt cache. And then we dont have to worry about bugs in the componentWillRecieveProps functions of each component which scare me. Now caching is separate from the components. I might have made some mistake in this change so let me check again.