search

WikiEducationFoundation / WikiEduDashboard

Wiki Education Foundation's Wikipedia course dashboard system
https://dashboard.wikiedu.org
MIT License
385 stars 600 forks source link

DiffViewer relies on a Revisions table query #5806

Open ragesoss opened 1 month ago

ragesoss commented 1 month ago

One of the unanticipated effects of deleting the Revisions table is that the DiffViewer feature is broken. It relies on a call to ArticlesController#details, which looks for a first revision and last revision of an edited article, and if those revisions are not found, there will be a JS error.

In this case, we should probably rework the frontend so that it does not rely on the Dashboard server to get the data it needs to render the diff from the earliest student revision to the latest one, perhaps by fetching the revision history of the article (within the date range of the course) and using that to find the earliest and latest revision.

ragesoss commented 1 month ago

@gabina this is relevant for you.

gabina commented 1 month ago

Took note of this :+1: