Misalignment on release 2.0.1

[ghandrew]

I think there is a misalignment between the release version 2.0.1 on npm and the one on github.

On npm is missing the new feature of "ttl" policy. As I can see the latest commit is missing.

Cheers.

[vegidio]

Came here to comment the same. The current version on NPM doesn't have the latest changes in master.

[vegidio]

I pulled the code from master branch and tried to make it work in my project without much success. Then I looked at the code and there's very evident bug in the implementation because it throws an exception whenever it tries to fetch a cached content.

Taking in consideration that this code was merged into master without the tests passing and with such a clear bug, I'm afraid that this project is abandoned. Which is a pity because it was a very useful package and I admire the amount of work that @timfpark put here. In any case, I forked the repo and corrected this bug.

If anyone is interested to use this corrected version then feel free to install it from my repo: yarn add vegidio-js/react-native-cache.

@timfpark if the project is not abandoned then please let me know and I would be happy to open a PR to have my fix merged and later have an update published on NPM 🙂

[eric-om]

@vegidio thank you for this!

[mickmaccallum]

Any update on this?

[jitenics]

I pulled the code from master branch and tried to make it work in my project without much success. Then I looked at the code and there's very evident bug in the implementation because it throws an exception whenever it tries to fetch a cached content.

Taking in consideration that this code was merged into master without the tests passing and with such a clear bug, I'm afraid that this project is abandoned. Which is a pity because it was a very useful package and I admire the amount of work that @timfpark put here. In any case, I forked the repo and corrected this bug.

If anyone is interested to use this corrected version then feel free to install it from my repo: yarn add vegidio-js/react-native-cache.

@timfpark if the project is not abandoned then please let me know and I would be happy to open a PR to have my fix merged and later have an update published on NPM 🙂

@vegidio there is promise warning as there is not gettime function found on line number 106 in cache.ts file, i am new to react native can you help me in that?

[vegidio]

@jitenics I will check this over the weekend