Page MenuHomePhabricator

Automatic topic subscriptions are not enabled for new users on their "non home wiki"
Closed, ResolvedPublic

Description

To avoid turning on this feature for "old" users, we set the default to be disabled/false/off.

We then have a hook that runs when an account is created to turn the preference on.

This hook also runs when an existing user logs into a 2nd wiki (or 3rd, 4th...).

We don't check at that point if the user is a new user who was defaulted into the feature, or an old user who we left the feature off for, instead we assume they are an old user and don't turn on the feature.

We copied this code from Echo, which also has a set of preference for new users, so this bug will be present there too.

QA
  • Create an account an en.wiki
  • Observe automatic topic subscriptions are enabled
    image.png (210×307 px, 15 KB)
  • Go to fr.wiki and look at your prefs
  • Observe automatic topic subscriptions are disabled
    image.png (220×352 px, 22 KB)
    Expected: enabled

Event Timeline

Change 943554 had a related patch set uploaded (by Esanders; author: Esanders):

[mediawiki/extensions/DiscussionTools@master] Enable automatic topic subscriptions for autocreate accounts

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

The above patch will enable topic subscriptions for "autocreate" accounts (accounts created when you login to a new wiki with an existing SUL account).

It is expected that when you login to a new wiki with an existing SUL account, that all of your preferences will be set to new defaults (excluding GlobalPreferences). I don't think this is any different. As an experienced user I would not be surprised to login to a new wiki and see completely different preferences and features enabled.

DLynch moved this task from Code Review to QA on the Editing-team (Kanban Board) board.
DLynch added a project: Editing QA.

Change 943554 merged by jenkins-bot:

[mediawiki/extensions/DiscussionTools@master] Enable automatic topic subscriptions for autocreate accounts

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

EAkinloose subscribed.

Tested with enwiki and dewiki

In prod since change isn't yet in prod

Screenshot 2023-08-09 at 01.38.29.png (388×1 px, 116 KB)

Result when tested in https://de.wikipedia.beta.wmflabs.org/

Screenshot 2023-08-09 at 01.40.35.png (348×1 px, 113 KB)