-
Notifications
You must be signed in to change notification settings - Fork 1.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Feature: Add clear image cache from memory and disk #425
Feature: Add clear image cache from memory and disk #425
Conversation
Codecov Report
@@ Coverage Diff @@
## master #425 +/- ##
==========================================
- Coverage 94.11% 86.36% -7.76%
==========================================
Files 1 1
Lines 17 22 +5
==========================================
+ Hits 16 19 +3
- Misses 1 3 +2
Continue to review full report at Codecov.
|
Codecov Report
@@ Coverage Diff @@
## master #425 +/- ##
==========================================
- Coverage 95.23% 86.95% -8.29%
==========================================
Files 1 1
Lines 21 23 +2
Branches 2 0 -2
==========================================
Hits 20 20
- Misses 1 3 +2
Continue to review full report at Codecov.
|
This is a pretty useful addition. Any thoughts on if it is going to be accepted? |
+1 on this, I'd love this addition to the library. |
2b79960
to
4ab44eb
Compare
Any expectations about when it will be merged? |
+1 would love this for development/testing |
Any updates on this? |
This is a pretty useful addition |
Why wasn't this merged? |
e0f48c3
to
28d06a5
Compare
@DylanVann I rebase the code and use promise api as you suggest please review it. |
dfe9aa2
to
ee3f590
Compare
ee3f590
to
1912be0
Compare
Any updates on this one ? |
Nice, Tks~ node_modules/react-native-fast-image/android |
So what is the situation with this PR? It seems that with the sheer quantity of issues concerning memory management with this lib, this is a quick win for those attempting to manage it manually, no? |
I'll try to create a PR which contains the newly updated docs |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great!
# [8.4.0](v8.3.7...v8.4.0) (2021-09-15) ### Bug Fixes * export FastImageStaticProperties ([#822](#822)) ([d69f692](d69f692)) * update dv-scripts ([61fab12](61fab12)) * update dv-scripts ([3c6d0f4](3c6d0f4)) ### Features * add clear image cache from memory and disk ([#425](#425)) ([818ed0c](818ed0c))
🎉 This PR is included in version 8.4.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Hello can anyone tell me how am I supposed to use it as simply calling FastImage.clearMemoryCache(); |
love u all |
@pranav-jft, u can call this method everytime app open, in useEffect of App.js... |
Is it possible to clear images with a TTL(Time to Live) like 10 days or something? |
# [8.4.0](DylanVann/react-native-fast-image@v8.3.7...v8.4.0) (2021-09-15) ### Bug Fixes * export FastImageStaticProperties ([#822](DylanVann/react-native-fast-image#822)) ([d69f692](DylanVann/react-native-fast-image@d69f692)) * update dv-scripts ([61fab12](DylanVann/react-native-fast-image@61fab12)) * update dv-scripts ([3c6d0f4](DylanVann/react-native-fast-image@3c6d0f4)) ### Features * add clear image cache from memory and disk ([#425](DylanVann/react-native-fast-image#425)) ([818ed0c](DylanVann/react-native-fast-image@818ed0c))
# [8.4.0](DylanVann/react-native-fast-image@v8.3.7...v8.4.0) (2021-09-15) ### Bug Fixes * export FastImageStaticProperties ([DylanVann#822](DylanVann#822)) ([d69f692](DylanVann@d69f692)) * update dv-scripts ([61fab12](DylanVann@61fab12)) * update dv-scripts ([3c6d0f4](DylanVann@3c6d0f4)) ### Features * add clear image cache from memory and disk ([DylanVann#425](DylanVann#425)) ([818ed0c](DylanVann@818ed0c))
I have a use case that need to clear image cache after user logout
I tested it on both android and iOS