search

ShaastraWebops / ERP

The Shaastra ERP is an Enterprise Resource Planner for IIT Madras' techfest, Shaastra.
8 stars 3 forks source link

Fix imports and clean up code #31

Open s-ramaswamy opened 12 years ago

s-ramaswamy commented 12 years ago

From what I have seen we seem to be importing everything in every page. Please ensure that only the modules you need are imported. Refrain from using from x import * .

There seem to be a lot of "empty" apps which don't do anything. Please remove these. Please remove all code which has been replaced by (bopefully) better code.

pandaops commented 12 years ago

this has already been somewhat done in the search branch. Will merge.

On Thu, Jul 19, 2012 at 2:39 PM, meetaw < reply@reply.github.com

wrote:

From what I have seen we seem to be importing everything in every page. Please ensure that only the modules you need are imported. Refrain from using from x import * .

There seem to be a lot of "empty" apps which don't do anything. Please remove these. Please remove all code which has been replaced by (bopefully) better code.

Reply to this email directly or view it on GitHub: https://github.com/ShaastraWebops/ERP/issues/31

Swapnil Basak, 2nd Year Undergraduate, Dept. of Electrical Engineering, IIT Madras