Page MenuHomePhabricator

AddLink: Skipped all dialog throws exception on ve.init.target.tryTeardown.then on mobile
Closed, ResolvedPublicBUG REPORT

Description

Steps to reproduce:

  1. use minerva skin
  2. skip all suggestions
  3. press "skip all"

On desktop, you'll see the post-edit dialog. On mobile, there's a console error logged indicating that ve.init.target.tryTeardown is not a promise. We need to use a different mechanism to switch to the read-only mode of the article.

Event Timeline

kostajh triaged this task as Medium priority.May 11 2021, 12:40 PM
kostajh changed the subtype of this task from "Task" to "Bug Report".
kostajh moved this task from Backlog to May 10 - May 14 on the Add-Link board.
kostajh added a subscriber: Etonkovidova.

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

[mediawiki/extensions/GrowthExperiments@master] PostEdit: Don't reset suggested edit session if task was cancelled

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

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

[mediawiki/extensions/VisualEditor@master] Fix ve.init.mw.MobileArticleTarget.tryTeardown return value

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

This should be fixed in VE but given its slower release cycle that won't happen in time. IMO we should just add some quick and dirty temporary solution like

-ve.init.target.tryTeardown( true, 'navigate-read' ).then( function () {
+( ve.init.target.tryTeardown( true, 'navigate-read' ) || $.Deferred().resolve() ).then( function () {

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

[mediawiki/extensions/GrowthExperiments@master] PostEdit: Fix skip all suggestions on mobile, and don't reset session if task was cancelled

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

Change 689088 abandoned by Kosta Harlan:

[mediawiki/extensions/GrowthExperiments@master] PostEdit: Don't reset suggested edit session if task was cancelled

Reason:

squashed

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

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

[mediawiki/extensions/GrowthExperiments@wmf/1.37.0-wmf.6] PostEdit: Fix skip all suggestions on mobile, and don't reset session if task was cancelled

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

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

[mediawiki/extensions/GrowthExperiments@wmf/1.37.0-wmf.7] PostEdit: Fix skip all suggestions on mobile, and don't reset session if task was cancelled

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

Change 689086 merged by jenkins-bot:

[mediawiki/extensions/GrowthExperiments@master] PostEdit: Fix skip all suggestions on mobile, and don't reset session if task was cancelled

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

Change 695038 merged by jenkins-bot:

[mediawiki/extensions/GrowthExperiments@wmf/1.37.0-wmf.6] PostEdit: Fix skip all suggestions on mobile, and don't reset session if task was cancelled

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

Change 695039 merged by jenkins-bot:

[mediawiki/extensions/GrowthExperiments@wmf/1.37.0-wmf.7] PostEdit: Fix skip all suggestions on mobile, and don't reset session if task was cancelled

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

Mentioned in SAL (#wikimedia-operations) [2021-05-26T11:26:29Z] <urbanecm@deploy1002> Synchronized php-1.37.0-wmf.6/extensions/GrowthExperiments/: GrowthExperiments backports (T283544; T282899; T282546) (duration: 01m 19s)

Mentioned in SAL (#wikimedia-operations) [2021-05-26T11:27:45Z] <urbanecm@deploy1002> Synchronized php-1.37.0-wmf.7/extensions/GrowthExperiments/: GrowthExperiments backports (T283544; T282899; T282546) (duration: 01m 06s)

Change 689269 abandoned by Kosta Harlan:

[mediawiki/extensions/VisualEditor@master] Fix ve.init.mw.MobileArticleTarget.tryTeardown return value

Reason:

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