⚓ T345257 Wire the reporting form to the API endpoint

	Subject	Repo	Branch	Lines +/-
	api: Support receiving "something else" details	mediawiki/extensions/ReportIncident	master	+22 -3
	form: POST form data to intake API	mediawiki/extensions/ReportIncident	master	+23 -3

Status	Subtype	Assigned	Task
Open		None	T337566 [EPIC]: Incident Reporting System - Minimal Testable Product (MTP)
Resolved		kostajh	T345257 Wire the reporting form to the API endpoint
Resolved		kostajh	T337635 Create API endpoint for receiving report data
Resolved		kostajh	T344997 Define email content and subject line
Resolved		kostajh	T345256 Send email based on form submission
Resolved		kostajh	T345813 Implement rate limits for submitting data to ReportIncident API
Resolved	BUG REPORT	kostajh	T347082 Form posts "something else" details, even if "something else" behavior is not checked

kostajh created this task.Aug 30 2023, 2:22 PM

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptAug 30 2023, 2:22 PM

kostajh added a subtask: T337635: Create API endpoint for receiving report data.Aug 30 2023, 2:22 PM

@Madalina @JSengupta-WMF What is the error message that we display to the user in case there is an error from the server-side? Do we allow the user to retry the submission? (My suggestion is to allow pressing "Submit" again.)

In T345257#9149651, @kostajh wrote:

@Madalina @JSengupta-WMF What is the error message that we display to the user in case there is an error from the server-side? Do we allow the user to retry the submission? (My suggestion is to allow pressing "Submit" again.)

I found this error message (https://drive.google.com/drive/u/0/folders/1H8doID3uD0rR_5O8iXF03vjSikseSDpd) which implies that the user has to start the process over again.

Change 955905 had a related patch set uploaded (by Kosta Harlan; author: Kosta Harlan):

[mediawiki/extensions/ReportIncident@master] form: POST form data to intake API

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

gerritbot added a project: Patch-For-Review.Sep 8 2023, 10:55 AM

Change 956421 had a related patch set uploaded (by Kosta Harlan; author: Kosta Harlan):

[mediawiki/extensions/ReportIncident@master] api: Support receiving "something else" details

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

kostajh moved this task from 🎬 Ready to 🔍 Review/Feedback on the Trust and Safety Tools Team Backlog (Kanban) board.Sep 19 2023, 11:28 AM

kostajh updated the task description. (Show Details)Sep 19 2023, 6:29 PM

In T345257#9151953, @kostajh wrote:

In T345257#9149651, @kostajh wrote:

@Madalina @JSengupta-WMF What is the error message that we display to the user in case there is an error from the server-side? Do we allow the user to retry the submission? (My suggestion is to allow pressing "Submit" again.)

I found this error message (https://drive.google.com/drive/u/0/folders/1H8doID3uD0rR_5O8iXF03vjSikseSDpd) which implies that the user has to start the process over again.

Let's follow up on this discussion in T338804: [S] Inform the user about the success or failure of submitting a report.

kostajh mentioned this in T346830: Unit test form component interaction with API.Sep 19 2023, 7:12 PM

Change 955905 merged by jenkins-bot:

[mediawiki/extensions/ReportIncident@master] form: POST form data to intake API

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

kostajh mentioned this in rEREIc7b1a710bd7d: form: POST form data to intake API.Sep 19 2023, 8:27 PM

ReleaseTaggerBot added a project: MW-1.41-notes (1.41.0-wmf.28; 2023-09-26).Sep 19 2023, 9:00 PM

I feel that showing user an error message that late asking to restart the process again could be frustrating to the user. I would keep the filled form open and show an inline error message with the option to resubmit it.

Change 956421 merged by jenkins-bot:

[mediawiki/extensions/ReportIncident@master] api: Support receiving "something else" details

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

kostajh mentioned this in rEREIf63d58e7b842: api: Support receiving "something else" details.Sep 21 2023, 2:35 PM

Maintenance_bot removed a project: Patch-For-Review.Sep 21 2023, 3:12 PM

In T345257#9181953, @JSengupta-WMF wrote:

I feel that showing user an error message that late asking to restart the process again could be frustrating to the user. I would keep the filled form open and show an inline error message with the option to resubmit it.

We will follow up on the workflows for that in T338804: [S] Inform the user about the success or failure of submitting a report

kostajh added a subtask: T347082: Form posts "something else" details, even if "something else" behavior is not checked.Sep 21 2023, 3:47 PM

I have verified the information listed in the ticket Description has been implemented and is working successfully at https://ko.wikipedia.beta.wmflabs.org/w/index.php?title=%EC%82%AC%EC%9A%A9%EC%9E%90%ED%86%A0%EB%A1%A0:Bluedot&uselang=en

Djackson-ctr updated the task description. (Show Details)Sep 25 2023, 7:55 PM

Djackson-ctr moved this task from 🐛 QA to 🤘 Done on the Trust and Safety Tools Team Backlog (Kanban) board.

kostajh closed subtask T347082: Form posts "something else" details, even if "something else" behavior is not checked as Resolved.Sep 29 2023, 11:44 AM

Madalina closed this task as Resolved.Oct 13 2023, 7:29 PM

Dreamy_Jazz closed subtask T337635: Create API endpoint for receiving report data as Resolved.Oct 25 2023, 1:02 PM