Page MenuHomePhabricator

Remove explicit "mailKey" from Owners packages
ClosedPublic

Authored by epriestley on May 4 2018, 8:44 PM.
Tags
None
Referenced Files
F15539620: D19427.id46474.diff
Fri, Apr 25, 3:05 AM
F15524767: D19427.id.diff
Mon, Apr 21, 10:14 AM
F15521328: D19427.diff
Sun, Apr 20, 1:09 PM
F15517030: D19427.id46468.diff
Fri, Apr 18, 11:39 PM
F15425365: D19427.diff
Mar 23 2025, 4:21 AM
F15424551: D19427.id46468.diff
Mar 22 2025, 10:59 PM
F15421674: D19427.id.diff
Mar 22 2025, 1:34 AM
F15420870: D19427.id46474.diff
Mar 21 2025, 6:26 PM
Subscribers
Restricted Owners Package

Details

Summary

Depends on D19426. Ref T13130. Ref T13065. While I'm making changes to Owners for "Ignore generated paths", clean up the "mailKey" column.

We recently (D19399) added code to automatically generate and manage mail keys so we don't need a ton of mailKey properties in the future. Migrate existing mail keys and blow away the explicit column on packages.

Test Plan

Ran migration, manually looked at the database and saw sensible data. Edited a package to send some mail, which looked good.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable