Change Details

Aim: explore the feasibility of comparing the html of the wikipedia page before and after reparsing with the wikidata change The prototype will compare the html from the cache (if available) and check if it changed, and insert the RC log accordingly (if diff is 'no change', suppress recent change log. If there is no cache to compare with, send the RC change log anyway.) Notes for reviewers: The prototype commits: - Core: https://gerrit.wikimedia.org/r/c/mediawiki/core/+/1260687 - Wikibase: https://gerrit.wikimedia.org/r/c/mediawiki/extensions/Wikibase/+/1261347 Things to consider: - will it not adversely affect e.g. page loading in production - can we inject asynchronous processes Acceptance criteria [x] Issue - recent changes not being written to the database (and/or mention it as an issue for the reviewers) [x] Check it's working locally - with no change, seeing that the recent change doesn't get inserted. Ideally, make a video showing it working [] There is a one-line that was added (some other ticket from the updated master) - we need to 'undo' it and leave a comment that it would need to be reconciled with our version [] Have a list of 'still todos' for the reviewers (including the ones above)

Aim: explore the feasibility of comparing the html of the wikipedia page before and after reparsing with the wikidata change The prototype will compare the html from the cache (if available) and check if it changed, and insert the RC log accordingly (if diff is 'no change', suppress recent change log. If there is no cache to compare with, send the RC change log anyway.) Notes for reviewers: The prototype commits: - Core: https://gerrit.wikimedia.org/r/c/mediawiki/core/+/1260687 - Wikibase: https://gerrit.wikimedia.org/r/c/mediawiki/extensions/Wikibase/+/1261347 Things to consider: - will it not adversely affect e.g. page loading in production - can we inject asynchronous processes Tradeoffs for the approach: - Around a third of pages have no cache for comparison. Therefore things would remain in the current state, of there being ‘some’ false positives. So this would only be able to solve the problem by a maximum of 66%. If the page is not in the cache, we can’t suppress the notification - False positives due to race conditions Acceptance criteria [x] Issue - recent changes not being written to the database (and/or mention it as an issue for the reviewers) [x] Check it's working locally - with no change, seeing that the recent change doesn't get inserted. Ideally, make a video showing it working [] There is a one-line that was added (some other ticket from the updated master) - we need to 'undo' it and leave a comment that it would need to be reconciled with our version [] Have a list of 'still todos' for the reviewers (including the ones above)

Aim: explore the feasibility of comparing the html of the wikipedia page before and after reparsing with the wikidata change The prototype will compare the html from the cache (if available) and check if it changed, and insert the RC log accordingly (if diff is 'no change', suppress recent change log. If there is no cache to compare with, send the RC change log anyway.) Notes for reviewers: The prototype commits: - Core: https://gerrit.wikimedia.org/r/c/mediawiki/core/+/1260687 - Wikibase: https://gerrit.wikimedia.org/r/c/mediawiki/extensions/Wikibase/+/1261347 Things to consider: - will it not adversely affect e.g. page loading in production - can we inject asynchronous processes Tradeoffs for the approach: - Around a third of pages have no cache for comparison. Therefore things would remain in the current state, of there being ‘some’ false positives. So this would only be able to solve the problem by a maximum of 66%. If the page is not in the cache, we can’t suppress the notification - False positives due to race conditions Acceptance criteria [x] Issue - recent changes not being written to the database (and/or mention it as an issue for the reviewers) [x] Check it's working locally - with no change, seeing that the recent change doesn't get inserted. Ideally, make a video showing it working [] There is a one-line that was added (some other ticket from the updated master) - we need to 'undo' it and leave a comment that it would need to be reconciled with our version [] Have a list of 'still todos' for the reviewers (including the ones above)