Page MenuHomePhabricator

Remove explicit "mailKey" from Owners packages
ClosedPublic

Authored by epriestley on May 4 2018, 8:44 PM.
Tags
None
Referenced Files
F18178379: D19427.id46474.diff
Sat, Aug 16, 1:29 AM
F18103370: D19427.id46468.diff
Sun, Aug 10, 4:47 AM
F18098037: D19427.diff
Fri, Aug 8, 5:58 PM
F17945398: D19427.id.diff
Thu, Jul 31, 12:03 PM
F17926880: D19427.diff
Wed, Jul 30, 1:47 PM
F17782395: D19427.id46474.diff
Jul 24 2025, 8:07 AM
F17763484: D19427.diff
Jul 22 2025, 11:27 PM
F17701373: D19427.id46468.diff
Jul 15 2025, 9:32 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