Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

(Add) Request #3676 #3742

Merged
merged 3 commits into from
Apr 11, 2024
Merged

(Add) Request #3676 #3742

merged 3 commits into from
Apr 11, 2024

Conversation

HDVinnie
Copy link
Collaborator

@HDVinnie HDVinnie commented Apr 8, 2024

- closes #3676
- replaces top_uploaders block with new top_users block which covers old stats plus more
Copy link
Collaborator

@Roardom Roardom left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In a future PR, it might be worth doing an optimization like the top torrents livewire component where only one panel is loaded at a time

@HDVinnie HDVinnie merged commit 27581be into 8.x.x Apr 11, 2024
10 checks passed
@HDVinnie HDVinnie deleted the Request-3676 branch April 11, 2024 22:27
@Roardom
Copy link
Collaborator

Roardom commented Apr 15, 2024

In a future PR, it might be worth doing an optimization like the top torrents livewire component where only one panel is loaded at a time

Benchmarked, and now we're above and beyond 30 second home page load times, way worse than before the major optimizations included in #3624

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.

[Request] Add More Tabs to top_uploaders.blade from Stats Page
2 participants