From e7fa62fc599b7583102fe48fbaad3db4f911a6da Mon Sep 17 00:00:00 2001 From: Michael Orlitzky Date: Sun, 29 Sep 2013 22:11:49 -0400 Subject: [PATCH] Move the LICENSE and README into a doc/ folder. Add a TODO. --- LICENSE => doc/LICENSE | 0 README => doc/README | 0 doc/TODO | 21 +++++++++++++++++++++ 3 files changed, 21 insertions(+) rename LICENSE => doc/LICENSE (100%) rename README => doc/README (100%) create mode 100644 doc/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. -- 2.44.2