Skip to content

Admin home page 128#259

Merged
DecDuck merged 26 commits into
Drop-OSS:developfrom
Pacodastre:admin-home-page-128
Nov 7, 2025
Merged

Admin home page 128#259
DecDuck merged 26 commits into
Drop-OSS:developfrom
Pacodastre:admin-home-page-128

Conversation

@Pacodastre
Copy link
Copy Markdown
Contributor

@Pacodastre Pacodastre commented Oct 8, 2025

This is what the new page looks like (style could be greatly improved here).

Screenshot 2025-10-08 at 17-16-39 Home - Admin - Drop

Some notes about this PR:

  • I refactored the sources table into its own component and modified it to add it to this page. Note that the table on the admin home page doesn't have edit/delete actions.
  • I added a new function on the file system providers fsStats which returns file system info for the base directory in the configuration of that provider.
  • The tooltip on the pie chart doesn't work properly. This is due to the fact that the component uses absolute/relative positioning for the link in the bottom right of the tile. I tried playing with the values a bit, but couldn't get it working properly. So either the tooltip will need to work differently, or the absolute/relative positioning needs to be removed from the tile component.
  • the pie chart svg uses absolute coordinates (rather than relative coordinates). I don't think it matters, but just wanted to point it out.
  • I still need to do a little bit of refactoring.

Here's what the sources page looks like now:

Screenshot 2025-10-08 at 17-17-00 Library Sources - Admin - Drop

Let me know what you want me to improve.

Fixes #128

Comment thread i18n/locales/en_us.json Outdated
@Pacodastre
Copy link
Copy Markdown
Contributor Author

Screenshot 2025-10-13 at 17-53-14 Home - Admin - Drop

Screenshot above after making some style changes

@Pacodastre
Copy link
Copy Markdown
Contributor Author

I added cache invalidation on signup/signin, which I forgot to do before.

@Pacodastre
Copy link
Copy Markdown
Contributor Author

This is what it looks like on the latest commit

Screenshot From 2025-11-03 23-35-58

Few details about the implementation:
Game sizes are cached 2 different ways:

  • gameid -> combined versions size
  • gameid -> versionName -> version size
    Sizes are recalculated when:
  • a game version is imported
  • a game version is deleted
  • a game is deleted

User count and activeUserCount are stored in the cache as well.
Their values are recalculated when:

  • a user is registered
  • a user is deleted
  • a client is revoked
  • a user signs in

@DecDuck DecDuck merged commit dfa30c8 into Drop-OSS:develop Nov 7, 2025
2 checks passed
@Pacodastre Pacodastre deleted the admin-home-page-128 branch November 7, 2025 23:43
Huskydog9988 pushed a commit to Huskydog9988/drop that referenced this pull request Apr 16, 2026
* First iteration on the new PieChart component

* Drop-OSS#128 Adds new admin home page

* Fixes code after merging conflicts

* Removes empty file

* Uses real data for admin home page, and improves style

* Reverts debugging code

* Defines missing variable

* Caches user stats data for admin home page

* Typo

* Styles improvements

* Invalidates cache on signup/signin

* Implements top 5 biggest games

* Improves styling

* Improves style

* Using generateManifest to get the proper size

* Reading data from cache

* Removes unnecessary import

* Improves caching mechanism for game sizes

* Removes lint errors

* Replaces piechart tooltip with colors in legend

* Fixes caching

* Fixes caching and slight improvement on pie chart colours

* Fixes a few bugs related to caching

* Fixes bug where app signin didn't refresh cache

* feat: style improvements

* fix: lint

---------

Co-authored-by: DecDuck <declanahofmeyr@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Admin home page

3 participants