Ishpaul Singh
Ishpaul Singh
> @janicduplessis @ishpaul777 any chance this could be stemming from this update? It appears that the root cause is stemming from the react-native-vision-camera upstream, so i dont think this comes...
Hey Thanks for bump i have this planned for today : )
Started android build to test existing proposals, i'll test and update in a while
Thanks for your proposals everyone! > For Android, we're already saving to Downloads folder (can be seen [here](https://github.com/Expensify/App/blob/db9e01cc302df7f2b3dd53a99c1082404676aedb/src/components/ClientSideLoggingToolMenu/index.android.tsx#L13-L20)), but the file path that we're showing is [of the localFile](https://github.com/Expensify/App/blob/db9e01cc302df7f2b3dd53a99c1082404676aedb/src/components/ClientSideLoggingToolMenu/index.android.tsx#L22), which...
> As of now, on Android file is getting saved to internal directory and then being copied to Downloads. My proposal directly saves it to Downloads, because of which the...
> But isn't it redundant to first download it in one folder and then copy to another? We can improve performance by directly downloading to Downloads folder. Also, logs can...
@ShridharGoel Do you have branch for your solution i'll test it again meanwhile
Why do we need a seprate function we can just use existing one with a param "saveToDownloads" and when true we save to downloads folder also why `LegacyDownloadDir` ?
Okay so after a bit of research i found this issue https://github.com/RonRadtke/react-native-blob-util/issues/294, so it looks like `LegacyDownloadsDir` uses depriciated apis under the hood and its not a future proof solution...
@ShridharGoel and @dominictb For profile trace tool in android, file is not saved in downloads folder so proposals are incomplete, Please include changes required to save it in downloads folder,...