Page MenuHomePhabricator

Watchlist Expiry: Static Watch Page (action=watch) confirmation should note length [medium]
Closed, DuplicatePublic

Description

As a watchlist expiry user, I want to view the selected temporary watch period in the success message (after selecting in in the static watch page), so that I can confirm that the correct period has been selected and saved.

This is a followup to T245565: Watchlist Expiry: Support for Static Watch Page (action=watch) [medium], since the acceptance criteria for that task didn't include changing the messages.

Background: As a user, when I visit https://en.wikipedia.beta.wmflabs.org/w/index.php?title=Main_Page&action=watch and I choose to watch for 1 week. I hit Watch. I should be told that the page has been added to my watchlist for 1 week. Currently, addedwatchtext is defined as "[[:$1]]" and its discussion page have been added to your [[Special:Watchlist|watchlist]]. The confirmation page should include the duration the user entered.

Acceptance Criteria:

  • Update the success message to state the length of time of the watch period
    • If permanent or temporary
      • [PAGE NAME] and its talk page have been added to your watchlist.
      • Watchlist Period: [[watch period]]

Visual Example (current behavior --> this should be modified):

Screen Shot 2020-04-14 at 4.39.03 PM.png (190×799 px, 28 KB)

Event Timeline

ifried added a subscriber: ifried.

Good catch, @DannyS712! The success message was not a part of the acceptance criteria of the original ticket (T245565), so this work would need to be handled in a separate ticket (such as this one). I'll update this ticket to specifically focus on the success message criteria.

ifried updated the task description. (Show Details)
ifried updated the task description. (Show Details)
ifried updated the task description. (Show Details)
ARamirez_WMF renamed this task from Watchlist Expiry: Static Watch Page (action=watch) confirmation should note length to Watchlist Expiry: Static Watch Page (action=watch) confirmation should note length [medium].Apr 14 2020, 11:30 PM

This issue is being addressed in T259055, which is currently in development, so I'll be closing this ticket as a duplicate.