> The module looks alright, but needs testing. I have a few minor > coding/style comments, but that can be addressed later or during > importing it.
Yeah, I tried to keep to the style, but I have to admit a couple of things go against deeply-ingrained habits, so I wouldn't be surprised if I missed some some stuff. > Michael, is the attached code the latest that you have, or have > you done any changes later? I haven't done any more development at this point. Though as I noted in another mail, some more development may be warranted---it did not perform as expected when I set it up in production. I anticipate this being relatively minor to correct, since it seems to work well in testing. > Do we have a licensing agreement from Michael? You should---I emailed a scan of a completed and signed form to [email protected] more than a week ago, and there should also be a corporate assignment for me from ironicdesign.com that was faxed in even before that. If either of those are missing, please let me know---we didn't get any ack for either. > I'd suggest we go on with importing the module - > - as it is an optional backend module what is important is that > we have an intention of adopting it and that it compiles cleanly, > any minor bugs can be fixed along the way. That would be very cool. I certainly intend to keep working on it, but I would also appreciate any input others have. Mike.
