eupgrade: set cached_only to false

[selankon]

Fixes #1069

[spiccinini]

Did you try with the lime-app? Forcing no cache will slow the load of the app right @germanferrero ?

[selankon]

@spiccinini the react query that check for updates should not block the ui.

BTW, we could use a background process or something like that, to check the updates, and let this just set to false? I am not familiarized with OWRT systems, have they something similar to services?

[G10h4ck]

missing new line at the end of file

[G10h4ck]

I would expect from a function named like that to not return a cached result.
Do the sub-called function trigger the download?
If this change slow down the UI, I would suggest to split triggering the download into another function

[spiccinini]

Do the sub-called function trigger the download?

AFAIK no. The issue is that until the http GET does not return it will block (and if you don't have internet it will timeout in more than a few seconds).

[spiccinini]

@spiccinini the react query that check for updates should not block the ui.

You mean that the current code does not block or that it should not? In some version of the lime-app it blocked the main page of the app, if you tested something like even adding a sleep(30) in the lua file does not block then good!.

[ilario]

This edit has been included by @selankon in #1101 so I would close this pull request.
If you consider that the discussion was not finished and that the change should be reverted, please re-open this or open a new issue.