Page MenuHomePhabricator
Differential D17786

Migrate accounts to correct user email verification state flag
ClosedPublic
Actions

Authored by epriestley on Apr 24 2017, 4:25 PM.
Tags
None
Referenced Files
F14010827: D17786.id42763.diff
Thu, Oct 31, 12:44 PM
F14010826: D17786.id42766.diff
Thu, Oct 31, 12:44 PM
F14010651: D17786.diff
Thu, Oct 31, 11:32 AM
F14008546: D17786.id.diff
Wed, Oct 30, 12:11 AM
F13978541: D17786.diff
Fri, Oct 18, 11:43 PM
F13978037: D17786.id42763.diff
Fri, Oct 18, 9:15 PM
Unknown Object (File)
Oct 10 2024, 10:02 AM
Unknown Object (File)
Oct 6 2024, 9:56 AM
View All Files
Subscribers
None

Details

Reviewers
chad
Maniphest Tasks
T12635: Can't verify primary emails for existing users
Commits
rPe2a94019b1f1: Migrate accounts to correct user email verification state flag
Summary

Depends on D17785. Fixes T12635. There was a bug where users could verify their primary email without getting the "isEmailVerified" flag set on their accounts.

D17785 fixes this bug. This change migrates affected account to fix their state, now that they can't get in trouble any more (hopefully).

Test Plan
  • Explicitly removed this flag from a bunch of accounts.
  • Ran migration, saw the accounts get fixed.
  • Ran migration again (storage upgrade --apply ...), saw the accounts not get touched.
  • We have 117 affected accounts on secure, so I'll verify that this fixes them.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Revision Contents
Changeset List

PathSize
resources/
sql/
autopatches/
34 lines

Diff 42766

View Options

resources/sql/autopatches/20170424.user.01.verify.php

Loading...
Phabricator · Built by Phacility