Page MenuHomePhabricator

Temp accounts shouldn't receive Growth notifications
Closed, ResolvedPublic

Description

Temporary accounts will receive notifications, but they don't have access to adjust preferences.
Temp accounts should keep the default notification preference (to keep it simple and because then we aren't adding as much to the user_properties table bloat)

However, Temporary accounts won't have access to the newcomer homepage and suggested edits, so we don't want them to receive Growth notifications.


Given an account is a Temporary account,
Then it shouldn't receive Growth notifications

Event Timeline

KStoller-WMF added a subscriber: Urbanecm_WMF.

This should already happen: if homepage isn’t enabled, Growth notifications shouldn’t get triggered. So I'm moving this to QA.

@Urbanecm_WMF - Is this task ready for testing, or are we waiting for more patches related to T341976: IP Masking: Change User::isRegistered() and User::isAnon() with User::isNamed in Growth-managed extensions before testing?

@Urbanecm_WMF - Is this task ready for testing, or are we waiting for more patches related to T341976: IP Masking: Change User::isRegistered() and User::isAnon() with User::isNamed in Growth-managed extensions before testing?

This should be ready for testing, GrowthExperiments is supposed to be fully IP-Masking ready for now. T341976 will receive patches related to other Growth-maintained extensions, but that won't affect this task at all.