Page MenuHomePhabricator

Migrate accounts to correct user email verification state flag
ClosedPublic

Authored by epriestley on Apr 24 2017, 4:25 PM.
Tags
None
Referenced Files
F13316406: D17786.diff
Thu, Jun 13, 3:36 AM
F13309780: D17786.id42763.diff
Mon, Jun 10, 10:35 AM
F13298381: D17786.diff
Fri, Jun 7, 6:42 AM
F13274157: D17786.id42763.diff
Fri, May 31, 3:21 AM
F13274155: D17786.id42766.diff
Fri, May 31, 3:21 AM
F13274152: D17786.id.diff
Fri, May 31, 3:21 AM
F13271348: D17786.diff
Wed, May 29, 8:47 PM
F13235548: D17786.diff
Tue, May 21, 5:52 AM
Subscribers
None

Details

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
Branch
verify2
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 16690
Build 22262: Run Core Tests
Build 22261: arc lint + arc unit