-
Notifications
You must be signed in to change notification settings - Fork 471
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 the /stats module #3712
Comments
There are a few things, like the MeetingRegistration model, that should have been in other apps to begin with that will have to be moved. |
Tero mentioned at https://mailarchive.ietf.org/arch/msg/tools-discuss/0Tw_KT4pYdUxoIf36p8yzn_bvEM that some parts of the existing implementation may remain useful, and we should review whether/what to move into the relevant apps while waiting for re-implementation. |
Are you still interested in a PR that rips out the document stats? |
Yes, and the meeting stats. It would be best if the same PR moved the review-team stats into the review application. |
The |
The /stats module as it currently exists is an attractive nuisance. It was intended to move the nice things that Jari has been maintaining into the datatracker, but the implementation didn't really achieve that.
As it is, what's presented ranges from incomplete to actually wrong.
We're going to remove it and schedule a new effort to build better statistics.
The text was updated successfully, but these errors were encountered: