Expenso-iOS icon indicating copy to clipboard operation
Expenso-iOS copied to clipboard

Delete previous attachment, when editing transaction

Open sameersyd opened this issue 3 years ago • 5 comments

The app holds memory of previous images attached in a transaction even after deleting or updating them with new one.

  • Delete the previous images from memory so the app size can decrease

sameersyd avatar Feb 17 '21 04:02 sameersyd

Hey @sameersyd are you working in this? Or is it available for people to contribute? If yes, let me know. I'd like to work on this. Thanks!

tejuamirthi avatar Feb 27 '21 00:02 tejuamirthi

Hello @tejuamirthi, Yes this project is open for contribution. I'll assign you for this issue.

sameersyd avatar Feb 27 '21 03:02 sameersyd

So, @sameersyd Are you sure about the app storing and not relieving the memory of the images deleted/updated? If that is the case, they must be show in transactions right? As far as I can see about this and the issue I can think of is, the memory allocated for the image might be open for re-use, but not cleared. That did answer my questions. I'd love to hear your thoughts. WDYT? Sorry for getting back late, thanks!

tejuamirthi avatar Mar 11 '21 17:03 tejuamirthi

@tejuamirthi When a transaction is deleted, the app deletes core-data object but the image in CD Object is still there, holding the disc memory, thus not decreasing the app size. You can either use instruments or directly look into simulator storage using path, the app will have all the images incl. deleted transaction's.

sameersyd avatar Mar 13 '21 06:03 sameersyd

Hii can you assign me this issue ?

narendrajagne avatar Oct 04 '23 10:10 narendrajagne