Maniphest T203783

Rewrite ext.flaggedRevs.review.js to make use of action review
Closed, DeclinedPublic
Actions

Assigned To

None

Authored By

	TheDJ
	Sep 7 2018, 11:59 AM

Description

This module still uses the ajax dispatcher call RevisionReview::AjaxReview, but can easily be rewritten to use action=review

Details

	Subject	Repo	Branch	Lines +/-
	AjaxDispatcher: Rewrite to use API instead	mediawiki/extensions/FlaggedRevs	master	+123 -136

Customize query in gerrit

Related Objects
Search...

Status	Assigned	Task
Resolved	matmarex	T45175 MediaWiki AJAX interface (tracking)
Resolved	daniel	T42787 Remove legacy ajax interface
Open	None	T31145 Core features that shouldn't be in core (tracking)
Resolved	Ladsgroup	T42786 Remove AjaxDispatcher from MediaWiki core
Resolved	Vlad.shapik	T94006 Remove AjaxDispatcher use from FlaggedRevs
Declined	None	T203783 Rewrite ext.flaggedRevs.review.js to make use of action review

Event Timeline

TheDJ triaged this task as Medium priority.Sep 7 2018, 11:59 AM

TheDJ created this task.

Arkadiy_Voronov subscribed.Oct 9 2018, 6:42 AM

Not sure if this would be impacted by T87457: Add param for the old stable revision id in api action=review

Change 467452 had a related patch set uploaded (by TheDJ; owner: TheDJ):
[mediawiki/extensions/FlaggedRevs@master] AjaxDispatcher: Rewrite to use API instead

https://gerrit.wikimedia.org/r/467452

gerritbot added a project: Patch-For-Review.Oct 15 2018, 7:46 PM

Liuxinyu970226 added a project: Growth-Team.Nov 21 2018, 2:31 PM

Change 467452 abandoned by TheDJ:
AjaxDispatcher: Rewrite to use API instead

https://gerrit.wikimedia.org/r/467452

TheDJ unsubscribed.Jan 29 2020, 10:15 PM