Is this the one we talked about and said we'd change to rounding system when we display "days left"?

I think we said that we'd round to the nearest whole number of days (e.g. 2 days 15 hours would be rounded to 3 days). Is that correct? Or do we only want to do this for 6/7 days? If there's less than 12 hours to go should it say "1 day"?

@Samwilson Yes, this is the one! We want round up, so after a user watches a page for "one week," they see "7 days" rather than "6 days." Quick question, now that I think about it -- is there a reason why it was proposed to round up for 12 hours? Wouldn't it make more sense to round up for just under 24 hours?

Also, great question about if it should apply to only 6/7. While the priority is to fix the 6/7 issue, we can also have a general rule for anything under 1 week. If someone is watching a page for 48 more hours, I think it makes sense if it says "2 days left." If someone is watching a page for 50 hours, I think it can say "3 days left" (since it's technically more than 2 days). If less than 12 hours are left, it may be misleading to say "1 day left," since one can easily assume that they have around 24 (or at least 12!) hours left. However, the other alternative is to display "XX hours left" (which could be even more misleading, since we can't actually give the correct number of hours) or "0 days left," which could look weird/confusing ("Why am I still watching the page if i have 0 days left?"). This could go in a few directions, so I'm pinging @Prtksxna for his input. Thanks in advance :)

@Prtksxna and I discussed this:

• We both agree that general day periods should be expressed in 24 hours periods, generally. So, for example, if I watch a page for 1 week, I should see "7 days" for about 24 hours.
• If the page is being watched for less than 1 days, we both agree that i should not say "0 days" or "XX hours." Instead, it can say "Less than 1 day left" or "A few hours left." Ilana prefers "A few hours left," but we can discuss this as a team.

Change 604198 had a related patch set uploaded (by Scardenasmolinar; owner: Scardenasmolinar):
[mediawiki/core@master] Add getExpiryInDays function in WatchedItem

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

Change 604198 merged by jenkins-bot:
[mediawiki/core@master] Add getExpiryInDays function in WatchedItem

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

We appear to be rounding up the number of days left until expiry.

For example:

I assume the "0 days left" will be changed in T255632.

Tested on https://wikidata.beta.wmflabs.org MediaWiki 1.35.0-alpha (6c57976) 2020-06-22T06:41:20.

For regression purposes, on my local environment, I disabled watchlist expiry and retested action=watch. It worked fine and there were no errors in the logs.