-
Notifications
You must be signed in to change notification settings - Fork 660
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
IndividualStatisticsPresenter and user profile page rely on Revisions table #5995
Comments
I will work on this |
@ragesoss I would also like to work on this. After looking through the IndividualStatisticsPresenter query logic responsible for extracting the data, I was thinking if it was okay for the statistics section of the dashboard to now focus on articles created by a student rather than those edited (i.e. revisions). |
@ragesoss Sir, for removing revisions table dependency there were two approaches: Strategy 1: Create New Contribution Tracking
Strategy 2: Remove Feature
Conclusion
Kindly share your thoughts on which approach would be best to follow. |
@omChauhanDev I think Strategy 2 is the best option at this point. We can look into an efficient way to replace it with similar functionality later on, perhaps, but it's not critical. |
We are deprecating the Revisions table and have removed old revisions from the production databases. However, the user profile pages — particularly, the feature for showing 'contributions as a student' — still relies on data from this table.
We need to either update the way this works so that it does not rely on the Revisions table (which may mean displaying a more limited amount of stats) or just remove this feature entirely.
The text was updated successfully, but these errors were encountered: