Page MenuHomePhabricator

hCaptcha: Add support for hCaptcha in the MobileFrontend editor
Open, In Progress, Needs TriagePublic10 Estimated Story Points

Description

Summary

  • MobileFrontend uses the API to edit
  • When an edit is made using the MobileFrontend, the edit should be prevented unless the user solves an hCaptcha challenge

Acceptance criteria

  • An hCaptcha captcha challenge is triggered when the user edits a page using the mobile frontend
  • When the challenge is shown, the edit is not made unless the user correctly solves it
  • The captcha challenge should be shown in the same screen used for providing the edit summary (instead of having an additional step as it happens for other captcha types)
  • If the user clicks outside the popup that contains the captcha, it goes back to the form that prompts for the edit summary; trying to submit the edit again will show the captcha again

Event Timeline

hector.arroyo renamed this task from hCaptcha: Add support for hCaptcha in the MoileFrontend to hCaptcha: Add support for hCaptcha in the MoileFrontend editor.Wed, Oct 15, 2:18 PM
hector.arroyo added a project: MobileFrontend.
hector.arroyo moved this task from Untriaged to MobileFrontend (Editor) on the MobileFrontend board.
hector.arroyo renamed this task from hCaptcha: Add support for hCaptcha in the MoileFrontend editor to hCaptcha: Add support for hCaptcha in the MobileFrontend editor.Wed, Oct 15, 2:21 PM
hector.arroyo changed the task status from Open to Stalled.Fri, Oct 17, 9:45 AM

Change #1196889 had a related patch set uploaded (by Harroyo-wmf; author: Harroyo-wmf):

[mediawiki/extensions/MobileFrontend@master] WiP Currently not working, requires previous patches to be merged

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

It is getting really confusing to work on this feature without having the previous patches merged first

The current solution has just been downvoted by the Editing team, raising the estination for this task.

hector.arroyo changed the point value for this task from 1 to 6.Tue, Oct 21, 3:25 PM

Change #1198289 had a related patch set uploaded (by Harroyo-wmf; author: Harroyo-wmf):

[mediawiki/extensions/MobileFrontend@master] editor: Add hooks required for supporting hCaptcha on edits

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

The solution has been reworked.

hector.arroyo changed the point value for this task from 6 to 10.Fri, Oct 24, 3:37 PM

Change #1196056 had a related patch set uploaded (by Harroyo-wmf; author: Harroyo-wmf):

[mediawiki/extensions/ConfirmEdit@master] hCaptcha: Support hCaptcha when using the MobileFrontend

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

hector.arroyo changed the task status from Stalled to In Progress.Tue, Oct 28, 4:40 PM

There's a change pending to be done in https://gerrit.wikimedia.org/r/c/mediawiki/extensions/ConfirmEdit/+/1196056 to make the popup with the challenge close when the user clicks outside it.

I'm updating the ACs to reflect this.

Change #1196889 abandoned by Harroyo-wmf:

[mediawiki/extensions/MobileFrontend@master] editor: Add support for hCaptcha

Reason:

Superseded by I09fa5d5d00755425b1ac3e56d9714a56333ffef6

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