John Hampton wrote:
Noah Kantrowitz wrote:Given that stream filters are now available, I would rather see people use those to alter the UI before adding such single-case extension points. You can use a request filter to handle the verification step.I disagree. I haven't looked at this specific case, so I'm not speaking directly to #874, but just because we have a general solution in stream filters, I don't think that precludes a more specific solution. Stream filters are great, but can also be a big pain in the rear. Sure, it's great to have an all-purpose tool, but it nicer to have a full toolbox.
Then the registration process in AccountManager should be reworked entirely to be a stateful system. An API similar (or even exactly the same) as the ticket workflow system could be added but for registration workflow. I would fully support that as a good, generic solution too, it just takes more effort. Adding hooks all over AccountManager will end up being a losing proposition eventually, changing the underlying model would be better in the long run in terms of flexibility.
--Noah
signature.asc
Description: OpenPGP digital signature
