Skip to content
This repository has been archived by the owner on May 7, 2024. It is now read-only.

System Download Limit Handling #5

Merged
merged 2 commits into from
Oct 16, 2023
Merged

Conversation

pprp
Copy link
Contributor

@pprp pprp commented Oct 15, 2023

Due to system constraints that permit the download of only up to 330 files at once, we've introduced a global variable named REQUEST_COUNT. If the file count exceeds 330, this variable helps determine if a 15-minute wait is necessary before proceeding with additional downloads.

@fedebotu
Copy link
Owner

fedebotu commented Oct 16, 2023

Hi there and thanks for your contribution!

I accidentally merged the previous PR believing it was a way to bypass the download limit. I think we should be a little careful about this. If a user tries downloading and the limit is hit, the app will just hang without any alert. The more than 10-minute wait can make it seem like a bug, when in fact it is not; I doubt a user would wait around 10 minutes thinking everything is working fine.

This should be either an option or it should be easy to see the reason why (e.g. in form of a popup) with maybe some extra info, such as using a VPN can bypass the problem

@fedebotu fedebotu merged commit f60f7f7 into fedebotu:main Oct 16, 2023
2 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants