← Back to team overview

launchpad-reviewers team mailing list archive

Re: [Merge] lp:~jtv/launchpad/bug-876348 into lp:launchpad

 

>From IRC (times in UTC):

(13:36:37) bigjools: jtv: actually EarlyReturn is not ideal thinking about it
(13:36:37) bigjools: we just need to add a rejection and let processing continue so it collects all the errors
(13:37:19) bigjools: I really detest the upload processor
(13:47:51) jtv: bigjools: I guess it's two problems: one, “malformed email address and policy that says to create the user” does something very different from “unknown email address and policy that says not to create the user” or “malformed email address and policy that says not to create the user.”  Two, it'd be nice to have more robust handling of these error conditions.
(13:47:59) jtv: Does that make sense?
-- 
https://code.launchpad.net/~jtv/launchpad/bug-876348/+merge/79954
Your team Launchpad code reviewers is requested to review the proposed merge of lp:~jtv/launchpad/bug-876348 into lp:launchpad.


References