Skip to content

Conversation

ahmethakanbesel
Copy link
Contributor

@ahmethakanbesel ahmethakanbesel commented Jul 3, 2023

Resolves #479
Local files used to be overwritten even if downloads failed.
Solved it by creating a temporary file and renaming it with the original filename after completing the download successfully.

@ahmethakanbesel ahmethakanbesel requested a review from a team as a code owner July 3, 2023 14:50
@ahmethakanbesel ahmethakanbesel requested review from ilkinulas and sonmezonur and removed request for a team July 3, 2023 14:50
sonmezonur
sonmezonur previously approved these changes Jul 6, 2023
@sonmezonur
Copy link
Member

@ahmethakanbesel Thanks for your PR. Could you please link your PR to the issue? (when PR merged it will automatically close the referenced issue) (docs)

@sonmezonur sonmezonur self-requested a review July 6, 2023 11:55
sonmezonur
sonmezonur previously approved these changes Jul 7, 2023
Co-authored-by: İlkin Balkanay <ilkinulas@gmail.com>
@sonmezonur sonmezonur merged commit 6ac390f into peak:master Jul 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Local file is lost even if the download fails
4 participants