Skip to content
This repository was archived by the owner on Jan 30, 2026. It is now read-only.

fix: backup download#1875

Open
flaviomoceri wants to merge 3 commits intodevfrom
fix/backup-download
Open

fix: backup download#1875
flaviomoceri wants to merge 3 commits intodevfrom
fix/backup-download

Conversation

@flaviomoceri
Copy link
Contributor

No description provided.

@flaviomoceri flaviomoceri self-assigned this Sep 25, 2025
@flaviomoceri flaviomoceri added the bug Something isn't working label Sep 25, 2025
@flaviomoceri flaviomoceri linked an issue Sep 25, 2025 that may be closed by this pull request
@catch-21
Copy link
Collaborator

I'm unable to reproduce this problem, even using a profile that has almost 200 posts including a few large files.

Before this fix, requests are all fired off concurrently which may cause strain and result in error for some that have too many requests to handle (client side?)
The fix requests sequentially, which is probably a more reliable approach.
Sev doesn't have error with the new approach but unsure whether data is complete.

The only change I suggest is to fail hard when network error occurs after the retries because otherwise the user will get only some of their data. Best they know it is incomplete with an error so we can easily know if there is a problem.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: Unable to download profile data from Settings

2 participants