X-Git-Url: http://gitweb.michael.orlitzky.com/?p=mailshears.git;a=blobdiff_plain;f=doc%2FTODO;h=6573397d83536b4e44b3417b77ac30833d314490;hp=8d773983a2cb247cdcb328638538a53fc9a23de5;hb=ef1b9dcb21d2e084408de19d844d438b99a00dc2;hpb=e7fa62fc599b7583102fe48fbaad3db4f911a6da diff --git a/doc/TODO b/doc/TODO index 8d77398..6573397 100644 --- a/doc/TODO +++ b/doc/TODO @@ -1,21 +1,5 @@ -* When we delete a domain, do we delete all of the accounts, too? Or - just the domain? We can do this in either the runner or the plugin. - - At the moment, it's mixed. The RmRunner does it manually, but e.g. - the PostfixadminDbRm plugin also deletes the mailboxes when the - domain is deleted. - - One doesn't seem any better than the other, but maybe if you - consider that some of the plugins have no-ops for - delete_domain(). Those could be made to actually do something - (i.e. delete all matching account names). Then we could get rid of - the get_domain_accounts() or-whatever-it's-called methods. - -* PostfixadminDb can be made a plugin -- prune just won't work without - it. - * Error reporting sucks, and when a domain or account doesn't exist we should be able to say so. The describe_domain/account functions - should also work better. Depending on what we do in the first bullet - point, they could either return a list of names, or - NotImplementedError. + should also work better. For plugins that don't implement domains, + we can return a (count of?) list of accounts, or fall back to the + account deletion descriptions.