Comment on Django-bcryptparentComments−nicksergeant15yYou sir, are a machine. Steve started working on this 3 feet behind me a little less than 2 hours ago.−jezdez15yIt took him two hours to monkey patch a few methods?−steveloshOP15yActually the code was already written -- see the bottom of the README.It took about 30-40 minutes to:* Add the BCRYPT_ROUNDS setting.* Remember how to allow for an optional setting (`if foo in settings` and `settings.get(foo, default)` don't work).* Add the setup.py file.* Write a README.* Test with a real Django site to make sure nothing was broken.* Create and push to repos on BitBucket and GitHub.−Jasber15y> Remember how to allow for an optional setting (`if foo in settings` and `settings.get(foo, default)` don't work).Instead of: settings.get(foo, default) Try: getattr(settings, foo, default) This looks useful. Thanks.−steveloshOP15yGood idea, pushed!−nicksergeant15yI left a few minutes after he started. He very well might've been banging his head against other (client-related) issues within those two hours.
Comments
You sir, are a machine. Steve started working on this 3 feet behind me a little less than 2 hours ago.
It took him two hours to monkey patch a few methods?
Actually the code was already written -- see the bottom of the README.
It took about 30-40 minutes to:
* Add the BCRYPT_ROUNDS setting.
* Remember how to allow for an optional setting (`if foo in settings` and `settings.get(foo, default)` don't work).
* Add the setup.py file.
* Write a README.
* Test with a real Django site to make sure nothing was broken.
* Create and push to repos on BitBucket and GitHub.
> Remember how to allow for an optional setting (`if foo in settings` and `settings.get(foo, default)` don't work).
Instead of:
Try: This looks useful. Thanks.Good idea, pushed!
I left a few minutes after he started. He very well might've been banging his head against other (client-related) issues within those two hours.