Handle internet connectivity drops gracefully
Closed, ResolvedPublic
Actions

Assigned To

Authored By

	Soda
	Mar 12 2021, 1:47 PM

Description

Currently, when the Pagelist Widget is unable to connect to the internet and receive a response, it will show an error message along the lines of An unexpected error was detected. Please report this error to phabricator.wikimedia.org with logs from the console.

This a canned message that we use when we are unable to determine what caused the error.

Instead of this, we should have a custom message explaining that the pagelist cannot be fetched due to network issues.

Documentation

Tutorial on setting up and working with Gerrit (Gerrit is Wikimedia's code review platform)
File that needs to be modified
OOUI (the framework used to build the PagelistWidget)
MediaWiki Messages API
Guidelines on creating new messages

Criteria

Create a new message that explains that the pagelist cannot be fetched due to network issues.
Display the new message when fetching the pagelist errors out due to network issues.

Details

	Subject	Repo	Branch	Lines +/-
	Handle internet connectivity drops gracefully	mediawiki/extensions/ProofreadPage	master	+8 -1

Customize query in gerrit

Event Timeline

Soda created this task.Mar 12 2021, 1:47 PM

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptMar 12 2021, 1:47 PM

Soda moved this task from Backlog to Pagelist Widget on the ProofreadPage board.Mar 12 2021, 1:47 PM

Hi, I am a python developer and a second-year student. I would like to work on this issue can someone guide me, please?

@Hadiya8149 To solve the issue you basically need to add a special case to this function that checks if the error is an HTTP error and emits a message informing users that the pagelist couldn't be fetched due to network issues.

Also, I'd suggest setting up a MediaWiki development environment and installing the ProofreadPage extension to make debugging a lot easier.

Yash4357 claimed this task.Mar 14 2021, 2:23 PM

Change 673776 had a related patch set uploaded (by Yash9265; owner: Yash9265):
[mediawiki/extensions/ProofreadPage@master] Handle internet connectivity drops gracefully

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

gerritbot added a project: Patch-For-Review.Mar 21 2021, 4:36 PM

Change 673776 merged by jenkins-bot:
[mediawiki/extensions/ProofreadPage@master] Handle internet connectivity drops gracefully

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

ReleaseTaggerBot added a project: MW-1.36-notes (1.36.0-wmf.36; 2021-03-23).Mar 21 2021, 6:00 PM

Maintenance_bot removed a project: Patch-For-Review.Mar 21 2021, 6:10 PM

Soda closed this task as Resolved.Mar 27 2021, 10:22 AM