Paths

Table of Contentst

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

	F15505701: D17786.diff
	Tue, Apr 15, 5:05 AM

	F15477947: D17786.diff
	Mon, Apr 7, 7:05 PM

	F15449717: D17786.id42763.diff
	Fri, Mar 28, 11:50 AM

	F15445202: D17786.id42766.diff
	Thu, Mar 27, 12:50 PM

	F15441365: D17786.id.diff
	Wed, Mar 26, 5:50 PM

	F15438270: D17786.diff
	Tue, Mar 25, 11:55 PM

	F15397352: D17786.diff
	Mar 16 2025, 7:42 PM

	F15396595: D17786.diff
	Mar 16 2025, 3:17 PM

Subscribers

None

Details

Reviewers

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

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.Apr 24 2017, 4:25 PM

Harbormaster completed remote builds in B16690: Diff 42763.Apr 24 2017, 4:26 PM

chad accepted this revision.Apr 24 2017, 5:10 PM

This revision is now accepted and ready to land.Apr 24 2017, 5:10 PM

Closed by commit rPe2a94019b1f1: Migrate accounts to correct user email verification state flag (authored by epriestley, committed by epriestley). · Explain WhyApr 24 2017, 5:24 PM

This revision was automatically updated to reflect the committed changes.

epriestley mentioned this in T12635: Can't verify primary emails for existing users.Apr 24 2017, 5:27 PM

Revision Contents
Changeset List

Path

Size

resources/

sql/

autopatches/

20170424.user.01.verify.php

34 lines

Diff 42766

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

Loading...