Page MenuHomePhabricator

Migrate mailing lists to mailing list users
ClosedPublic

Authored by epriestley on Jun 2 2015, 10:19 PM.
Tags
None
Referenced Files
F13085124: D13128.diff
Wed, Apr 24, 11:42 PM
Unknown Object (File)
Sat, Apr 20, 3:19 AM
Unknown Object (File)
Thu, Apr 11, 7:59 AM
Unknown Object (File)
Mar 22 2024, 8:38 PM
Unknown Object (File)
Mar 19 2024, 6:15 AM
Unknown Object (File)
Feb 7 2024, 3:36 AM
Unknown Object (File)
Feb 6 2024, 1:56 AM
Unknown Object (File)
Feb 5 2024, 7:27 PM
Subscribers

Details

Summary

Ref T8387. This migrates lists in the database to users, and replaces all subscriptions.

This won't update Herald rules or saved search queries, but they're presumably rare and infeasibly complex to migrate.

Test Plan

This migration is relatively re-runnable, so I ran it a bunch of times with different setups using bin/storage adjust --apply. It successfully migrated lists into users and replaced them in all the places they were subscribed.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Migrate mailing lists to mailing list users.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
btrahan edited edge metadata.

This won't update Herald rules or saved search queries, but they're presumably rare and infeasibly complex to migrate.

How would these things break post killing mailing lists app? I am very comfortable with breaking them, I just would want users to be able to recover via the web ui.

This revision is now accepted and ready to land.Jun 3 2015, 11:08 PM

Break behavior for Herald should be clean at HEAD -- it got buffed up pretty substantially in D12957 for dealing with a similar case out of T7604.

Search will not find matching results, but shouldn't explode either.

This revision was automatically updated to reflect the committed changes.