Properly inform users when a page’s been moved
Closed, ResolvedPublicSpike
Actions

Assigned To

Authored By

	• scblr
	Feb 2 2021, 5:36 PM

Description

Derived from T269447#6796080, see @Sharvaniharan’s comment.

This is how it’s currently handled:

If the edit itself is not a regular edit [Moved, deleted, protected] that edit item will NOT be clickable. However, If it is a regular edit but the page is deleted, it will be clickable but will result in an error feedback shown to the user

This needs to be adressed in a better way. Design needs to look at proper UI copy to educate people and possibly support it with an illustration (similar to a 404 page on the Web)

The technical question to answer is: how do we detect if the page has been moved from only a specific watchlist entry (diff) that happened prior to the page having been moved?

Also, what kind of UX should we provide for the user to remove a page from their watchlist that has been moved?

Related Objects
Search...

		Status	Subtype	Assigned	Task
		Resolved		JTannerWMF	T273254 [EPIC] POST V2.7.33X-R-Rosgulla Initial Watchlist Release Clean Up
		Resolved	Spike	• Sharvaniharan	T273657 Properly inform users when a page’s been moved

Event Timeline

• scblr created this task.Feb 2 2021, 5:36 PM

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptFeb 2 2021, 5:36 PM

WaheedHafez subscribed.Feb 2 2021, 5:37 PM

This will be treated initially as a spike for investigation. (Actually development will depend on results of spike.)

Restricted Application changed the subtype of this task from "Task" to "Spike". · View Herald TranscriptFeb 4 2021, 5:40 PM

Dbrant updated the task description. (Show Details)Feb 4 2021, 5:42 PM

• Sharvaniharan claimed this task.Feb 8 2021, 11:37 PM

• Sharvaniharan moved this task from Ready for dev to Doing on the Wikipedia-Android-App-Backlog (Android Release FY2020-21) board.

Few notes from the spike:

When a page is 'Moved':
1.The particular edit that moved the page is grayed out and is not clickable:

However, the previous edits from that page are show in the list, but when we click them, we just show the common message that the page has been moved, but we do not attempt to get that diff details. This is because, for a moved page, the revision details are not returned correctly. For a regular page, when we request details about a particular revision[edit], api returns an olderRevisionId and a newerRevisionId, which represent the previous and next edits. However, for a moved page, the olderRevisionId is always returned as '0'. Since we know that it will result in a 404, we do not make the call at all.. So a user sees a screen like this: [which is slight improvement to what we see in the description of this task, where previously we were making the call anyway and it would result in a common error view being shown]

I have marked a test page for deletion and will update this ticket with findings once it is deleted.

We do not have any issues with deleted pages, as only the deleted edit is shown and is disabled and unclickable. So this spike is complete with following info:
For moved pages, the comment informs the users to where it was moved to , and no diff is shown. I also do not see a very obvious way for us to know that a particular 'regular edit' is of a moved page.
For deleted pages, the current behavior is sufficient.
@schoenbaechler @JTannerWMF Lmk if it looks good for now.

• Sharvaniharan moved this task from Doing to Design signoff on the Wikipedia-Android-App-Backlog (Android Release FY2020-21) board.Feb 11 2021, 6:59 PM

How about this (specific) example on my Watchlist @Sharvaniharan:

This page:

https://de.wikipedia.org/wiki/Muriel_Staub

Has been moved to:

https://de.wikipedia.org/wiki/Wikipedia:Enzyklopädie/Muriel_Staub

During the time I had it on my Watchlist. It now displays this error when I tap on it:

Screenshot_20210215-174736.png (2×1 px, 53 KB)

Video

Is there a way to mark this as MOVED as well? See Zeplin:

https://zpl.io/2jgwMY6

Thanks!

• scblr moved this task from Design signoff to Did not pass QA on the Wikipedia-Android-App-Backlog (Android Release FY2020-21) board.Feb 15 2021, 4:51 PM

@schoenbaechler This looks a bit different from what I have tested. We might need to discuss this on a call. Are you sure you were using the latest build for the video?

• Sharvaniharan moved this task from Did not pass QA to Design signoff on the Wikipedia-Android-App-Backlog (Android Release FY2020-21) board.Feb 18 2021, 7:39 PM

• scblr moved this task from Design signoff to QA signoff on the Wikipedia-Android-App-Backlog (Android Release FY2020-21) board.Feb 24 2021, 11:17 AM

I believe this is good to go on 2.7.50343-alpha-2021-02-26

JTannerWMF moved this task from Ready for PM signoff to Ready for release on the Wikipedia-Android-App-Backlog (Android Release FY2020-21) board.Mar 2 2021, 9:53 PM

• Sharvaniharan closed this task as Resolved.Mar 11 2021, 12:35 AM

	F34106136: Screenshot_20210215-174736.png
	Feb 15 2021, 4:50 PM

	F34085980: Screenshot_20210202-183404.png
	Feb 2 2021, 5:36 PM

	F34100087: moved.png
	Feb 10 2021, 11:57 PM

	F34100078: movedPage.png
	Feb 10 2021, 11:45 PM

	F34100082: moved.png
	Feb 10 2021, 11:45 PM

Properly inform users when a page’s been movedClosed, ResolvedPublicSpikeActions

Description

Related ObjectsSearch...

Event Timeline

Properly inform users when a page’s been moved
Closed, ResolvedPublicSpike
Actions

Related Objects
Search...