serverinfo icon indicating copy to clipboard operation
serverinfo copied to clipboard

Published 20 hours ago • verified publisher icon nextcloud

Added 7 and 30 day values to "Active Users" stats graph

Open akhil1508 opened this issue 4 years ago • 3 comments

  • Added 7 and 30 day values to the "Active users" stats graph (returned through "lib/SessionStatistics.php" and used in frontend in "js/script.js)
  • Modified unit tests accordingly to test that new values are correctly returned
  • Package "nextcloud/coding-standard" updated to "^0.4.0" from "^0.3.0"(PHPLint refuses to install correctly with 0.3.0 in the checks otherwise)

akhil1508 avatar Jan 11 '21 06:01 akhil1508

Ping

arnauvp avatar Jan 29 '21 14:01 arnauvp

The performance is already pretty bad on large instances. Most likely due share statistics but better safe than sorry.

J0WI avatar Feb 17 '21 14:02 J0WI

On our instance, the serverinfo/api/v1/info?format=json call takes up to 5.5 seconds. The DB query to fetch unique users in last 30 days takes ~400ms before it's cached.

arnauvp avatar Feb 19 '21 06:02 arnauvp

Needs a major overhaul after https://github.com/nextcloud/serverinfo/pull/420

Want to have a look?

nickvergessen avatar Jan 25 '23 15:01 nickvergessen