From: Michael Orlitzky Date: Mon, 30 Sep 2013 02:11:49 +0000 (-0400) Subject: Move the LICENSE and README into a doc/ folder. X-Git-Tag: 0.0.1~85 X-Git-Url: https://gitweb.michael.orlitzky.com/?a=commitdiff_plain;h=e7fa62fc599b7583102fe48fbaad3db4f911a6da;p=mailshears.git Move the LICENSE and README into a doc/ folder. Add a TODO. --- diff --git a/LICENSE b/doc/LICENSE similarity index 100% rename from LICENSE rename to doc/LICENSE diff --git a/README b/doc/README similarity index 100% rename from README rename to doc/README diff --git a/doc/TODO b/doc/TODO new file mode 100644 index 0000000..8d77398 --- /dev/null +++ b/doc/TODO @@ -0,0 +1,21 @@ +* 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.