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

Remove stats.js #44

Open
handstandsam opened this issue Feb 10, 2025 · 1 comment
Open

Remove stats.js #44

handstandsam opened this issue Feb 10, 2025 · 1 comment

Comments

@handstandsam
Copy link
Collaborator

We now have individual stats. We don't need to pull all of the data at once.

@handstandsam
Copy link
Collaborator Author

A lot of work was done regarding this in #43, and then some follow up commits that use that data instead of the full stats.js file. There are a couple pages that still haven't been refactored out of using all the stats so stats.js remains, but shouldn't be used in the report unless needed.

Long term goal is to remove it. It can be a VERY large file and splitting it up makes a lot more sense.

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

No branches or pull requests

1 participant