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

Revert "Turn off user site stats for the day" #1887

Merged
merged 1 commit into from
Feb 7, 2024

Conversation

mo-nathan
Copy link
Member

Reverts #1842

I need the user summary page to be working.

@coveralls
Copy link
Collaborator

Coverage Status

coverage: 94.555% (+0.05%) from 94.509%
when pulling 64a81ad on revert-1842-nimmo-user-nostats
into 14529b5 on main.

@nimmolo
Copy link
Contributor

nimmolo commented Feb 7, 2024

Maybe this is a good time to ask --
should I make the stats section a lazy loaded panel that people have to click on to reveal?

In a separate PR of course... this one is fine. I just dread clicking on any user! The load time is heinous.

The other idea I had was to use the cache the user stats for all users, for the week or maybe the day. Since we now have the refresh caches script working, thanks to @pellaea, that could be easier to implement.

@mo-nathan mo-nathan merged commit 83b0061 into main Feb 7, 2024
5 checks passed
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.

None yet

4 participants