Page MenuHomePhabricator

Return response for each contribution individually
Open, MediumPublicFeature

Description

Feature summary:
The response from api/post-contribution route should indicate the success/failure of each individual contribution (as multiple edits are performed one at a time).

We also need a way to indicate when an edit was not allowed (e.g. because it was on the Denylist)

So we should have the following data in the response for each edit:

status: (e.g. 200)
response: (response object from the edit, either success or fail data)
blocked: (true / false)
block_reason:  (text string)

The actual results will be an array of objects with the keys shown above.

NOTE: Another patch will be used to handle the new data in the front end and communicate the results to the end user

Event Timeline

NavinoEvans moved this task from Pending to Incoming features on the ISA board.

Change 998443 had a related patch set uploaded (by Juniorbesong; author: Juniorbesong):

[labs/tools/Isa@main] BUG: T323836. Updated isa/campaigns/routes.py BUG: T323836. Updated isa/campaigns/routes.py Change-Id: Id06ef3e2a564d604cc99ed06fbf7914c50f73dca

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

Juniorbesong changed the task status from Open to In Progress.Feb 7 2024, 4:18 PM
Aklapper changed the task status from In Progress to Open.Apr 11 2025, 10:20 PM
Aklapper subscribed.

Resetting task status from "In Progress" to "Open" as this task has been "in progress" for more than one year (see T380300). Feel free to set that status again, or rather break down into smaller subtasks.