[BUG] unable to change nb of days on Android

henrywhitaker3 / Speedtest-Tracker

Continuously track your internet speed

GNU General Public License v3.0

1.53k stars 121 forks source link

[BUG] unable to change nb of days on Android #773

Open toxic0berliner opened 3 years ago

toxic0berliner commented 3 years ago

Describe the bug A clear and concise description of what the bug is.

To Reproduce Steps to reproduce the behavior:

Go to homepage
Scroll down to bottom
Type 27 instead of the 7 days
Notif shows loading
Notif disappears, but keep waiting just in case
Still nothing, only 7 days are shown, 30 are in the history when looking at db

Context

OS: Docker
Speedtest-Tracker Version: 1.12.0
Firefox for Android or Chrome for Android have the same issue

Additional context Add any other context about the problem here.

benjamin-feron commented 3 years ago

Exactly same issue on my fresh Docker install but not only on Android but on chrome or firefox too. No console error and no network error on debugger.

benjamin-feron commented 3 years ago

I've solved my problem with a small code modification. File app/Http/Controllers/HomepageDataController.php, line 30 : replace 'time' => run(GetSpeedtestTimeData::class), with 'time' => run(GetSpeedtestTimeData::class, $days), I'm not a laravel user so I don't know if that is conventional at all.

quentinhayot commented 2 years ago

Same issue with Docker on a Synology NAS

dereitz commented 2 years ago

I am experiencing the same issue but in my case I'm using Google Chrome.

Is anyone maintaining this project any longer? Maybe dead?