Closed satindergrewal closed 9 years ago
I finished my review of the code.
See the comments. The changes mentioned need to be made before we can merge this. You can make the changes in another commit and push those, or override your existing commit and force-push the changes (that results in a slightly cleaner history).
Thanks, I'm new to Coffee Script. I did it with my limited understanding. Will try to make the changes according to your suggestions.
Closing this for now. If the issues are addressed and new commits are pushed we can re-open.
Gonna have a look at this later