Page MenuHomePhabricator

Watchlist Expiry: Support for Move [MEDIUM]
Closed, ResolvedPublicFeature

Description

As a Watchlist Expiry user, I want to be able to temporarily watch items via Move, so that I can access the same feature when using such functionality.

Acceptance Criteria:

  • Implement support for watchlist expiry drop-down in Move
    • A new label to the right (or above, depending on particular UI) of watchlist checkbox: "Watchlist time period:"
    • A drop-down to the right of label (inactive by default, before box checked)
  • When user checks the box to "Watch this page," the drop-down should become active
  • The user should be able to select watch period via drop-down
    • The drop-down has the following options:
      • Permanent (as default)
      • for 1 week
      • for 1 month
      • for 3 months
      • for 6 months
  • If the page has already been watched, the behavior should be as follows:
    • Drop-down should have days left as pre-selected option at top
    • The selection should be written as follows: "X days left" (for example, "10 days left" or "20 days left").
  • If the page has already been watched with a selected temporary time period, and the user clicks "Publish" without changing the time period selection, no changes to the watch period should be made.
  • The watch period should be saved or updated, if the user makes a new time period selection.

Visual Examples of how drop-downs should appear:

Temporarily_watch_a_page_via_edit,_mockup_example.png (212×763 px, 59 KB)
edit page variations.png (984×760 px, 72 KB)

Visual Examples of pages that we will update with new drop-downs:

Screenshot_2020-06-25 Preferences - Test Wikipedia.png (1×2 px, 381 KB)

Screenshot_2020-06-25 Change protection level for DmazaTest - Test Wikipedia.png (1×2 px, 184 KB)

Screenshot_2020-06-25 Delete DmazaTest - Test Wikipedia.png (1×2 px, 148 KB)

examplemovepage.png (609×1 px, 89 KB)

Event Timeline

When moving a page, the option is "Watch source page and target page". Will it be possible to have separate expiry times for the source and target pages? Or is that out of scope for this task?

@Rchard2scout Thanks for bringing up this question! The team is no longer working on watchlist expiry, since we have officially concluded the project. However, there may be opportunities to expand the feature in the future, and adding the option to select or update an expiry upon move is one example of this. We have no immediate plans to do this work, since the team needs to focus on the projects we are currently working on. But our team, another team, or a volunteer developer may be able to do this work in the future. We have this ticket written, in case someone would like to pick it up. Thank you again for this question!

AlexisJazz changed the subtype of this task from "Task" to "Feature Request".May 1 2021, 10:58 AM

Change #1141062 had a related patch set uploaded (by Bunnypranav; author: Bunnypranav):

[mediawiki/core@master] SpecialMovePage: Watchlist expiry when moving a page

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

Test wiki created on Patch demo by Bunnypranav using patch(es) linked to this task:
https://patchdemo.wmcloud.org/wikis/cf45b141e9/w/

@ifried Would you be willing to review this?

Hi @Bunnypranav, thanks for the ping! It's so cool that you worked on this, nearly 5 years after I wrote this ticket (!). I am no longer on the Community Tech team, which built & maintains Watchlist Expiry, but I'll ping some members of the team so they are aware and can determine next steps. @JWheeler-WMF @dmaza

Test wiki on Patch demo by Bunnypranav using patch(es) linked to this task was deleted:

https://patchdemo.wmcloud.org/wikis/cf45b141e9/w/

Bunnypranav added a subscriber: MusikAnimal.

Hey @JWheeler-WMF and @dmaza, all the issues raised by @MusikAnimal have been resolved. Would anyone of you be willing to review this?

Bunnypranav changed the task status from Open to In Progress.May 29 2025, 11:23 AM

Change #1141062 merged by jenkins-bot:

[mediawiki/core@master] SpecialMovePage: Watchlist expiry when moving a page

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

Change #1155378 had a related patch set uploaded (by Bunnypranav; author: Bunnypranav):

[mediawiki/core@REL1_44] SpecialMovePage: Watchlist expiry when moving a page

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

Change #1155378 abandoned by Bunnypranav:

[mediawiki/core@REL1_44] SpecialMovePage: Watchlist expiry when moving a page

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