Make on-demand download timeout configurable.

[SternI]

The 30 seconds timeout was insufficient for large on-demand downloads.
Timeout is now configurable via settings.

[4sval]

too targeted/too niche, I would rather see a try catch on c4p than another settings nobody will use

[h4lfheart]

too targeted/too niche, I would rather see a try catch on c4p than another settings nobody will use

I had to add a setting for this in Fortnite Porting for all api requests and in on-demand settings. You would be surprised by how many people with horrible wifi need it for small requests to go through, it wouldn't hurt to add.

[4sval]

I had to add a setting for this in Fortnite Porting for all api requests and in on-demand settings. You would be surprised by how many people with horrible wifi need it for small requests to go through, it wouldn't hurt to add.

well, of course, it’s for high res textures on fortnite, your users are the exact target of this PR 😄

if it would not hurt to add then @SternI rename it HTTP Request Timeout or something like this and use it for everything.

• default to 30
• it does not have to be a directory setting
• do not use Math.Max or use it once in a setter
• you don’t need a backing field in the settings viewmodel, just a simple integer control two way binded straight to the settings field
• you removed some stuff from the previous PR

[SternI]

I had to add a setting for this in Fortnite Porting for all api requests and in on-demand settings. You would be surprised by how many people with horrible wifi need it for small requests to go through, it wouldn't hurt to add.

well, of course, it’s for high res textures on fortnite, your users are the exact target of this PR 😄

if it would not hurt to add then @SternI rename it HTTP Request Timeout or something like this and use it for everything.

• default to 30
• it does not have to be a directory setting
• do not use Math.Max or use it once in a setter
• you don’t need a backing field in the settings viewmodel, just a simple integer control two way binded straight to the settings field
• you removed some stuff from the previous PR

Done

[4sval]

dont need this

[4sval]

use the correct input control

[4sval]

dont need this

[SternI]

What about this