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

	Unknown Object (File)
	Sat, Apr 13, 8:35 AM

	Unknown Object (File)
	Sun, Mar 31, 10:20 PM

	Unknown Object (File)
	Mar 18 2024, 8:23 AM

	Unknown Object (File)
	Mar 18 2024, 8:23 AM

	Unknown Object (File)
	Mar 16 2024, 12:26 AM

	Unknown Object (File)
	Mar 10 2024, 5:20 PM

	Unknown Object (File)
	Mar 10 2024, 4:55 PM

	Unknown Object (File)
	Jan 7 2024, 4:21 PM

View All 85 Files

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...