react-native-fast-image icon indicating copy to clipboard operation
react-native-fast-image copied to clipboard

Get image path from cache

Open alexandrumic opened this issue 5 years ago • 20 comments

A lot of people has request this feature (including me), so thanks to this PRs: https://github.com/DylanVann/react-native-fast-image/pull/351 by @compojoom and https://github.com/DylanVann/react-native-fast-image/pull/557 by @dolsem I managed to add a new static method getCachePath that is retrieving the local cached path of an image by image source.

This library is awesome and is working very well, so I think this kind of feature is needed for the situations when you need to make an offline-first app or need to pass the path of the images to other libraries (such as image galleries, image editors and so on)

alexandrumic avatar Aug 21 '20 13:08 alexandrumic

@DylanVann I think this functionality adds a lot of value to the library and it would be nice if this PR could be merged.

sbalay avatar Sep 28 '20 13:09 sbalay

@DylanVann Can you please look this over and merge? Or give perms to a contributor to do so? I believe this addresses the concerns you expressed about #351.

benlevy0 avatar Nov 14 '20 17:11 benlevy0

When will this PR be merged? Thanks

tavindev avatar Nov 28 '20 12:11 tavindev

any update ? 🙏

billouboq avatar Dec 22 '20 16:12 billouboq

@DylanVann This needs to be really added to the package because we use react-native-photo-editor in our project and in offline mode it needs the saved path of the image, and we don't want to download all the images again with another package to get just the local path when offline.

bakir97 avatar Dec 23 '20 20:12 bakir97

any updates?

SecretKeeper avatar Dec 24 '20 12:12 SecretKeeper

@DylanVann why is this is not merged yet?

Jalson1982 avatar Feb 16 '21 12:02 Jalson1982

@DylanVann Any update? This is an amazing feature and it would be amazing to be merged already :)

nemanja-sljivic avatar Feb 16 '21 14:02 nemanja-sljivic

Hey, don't want to rush anyone but please when will this be merged, this seems like a very important feature that will help everyone. Thanks.

papidb avatar Apr 07 '21 17:04 papidb

@DylanVann

gabrieldonadel avatar Apr 07 '21 19:04 gabrieldonadel

Can we merge this? Thanks

1uss1 avatar Apr 23 '21 11:04 1uss1

image

papidb avatar Apr 26 '21 19:04 papidb

will-smith-crying-meme

arzel avatar May 26 '21 01:05 arzel

image

zmanring avatar Jun 10 '21 18:06 zmanring

is there any chance to get this published?

manuelblum avatar Jun 15 '21 06:06 manuelblum

Waiting this feature released.

image

Can this cat help you publish this feature?

colus001 avatar Jun 22 '21 04:06 colus001

Any day?..)

VDetish avatar Nov 15 '21 20:11 VDetish

@alexandrumic cant you create a patch file for us to use meanwhile?? Much helpful if done

abhinav-official avatar Apr 15 '22 10:04 abhinav-official

Sad that the PR hasn't been accepted yet, still very much needed

nofacez avatar Oct 06 '22 09:10 nofacez

this is exactly importain update

fukemy avatar May 04 '23 04:05 fukemy