JayVora-SerpentCS / OdooEduERP

Education ERP based on Odoo!
http://www.serpentcs.com
151 stars 181 forks source link

made improvements in school module #94

Closed Arpit-SerpentCS closed 6 years ago

murtuzasaleh commented 6 years ago

@Arpit-SerpentCS Improve the README.rst file of a module. https://github.com/Arpit-SerpentCS/OdooEduERP/blob/11.0_school/school/README.rst

murtuzasaleh commented 6 years ago

@Arpit-SerpentCS Please add domain inside the search condition. https://github.com/Arpit-SerpentCS/OdooEduERP/blob/11.0_school/school/wizard/move_standards.py#L29

murtuzasaleh commented 6 years ago

@Arpit-SerpentCS Please add active_id inside the browse method. https://github.com/Arpit-SerpentCS/OdooEduERP/blob/11.0_school/school/wizard/teriminate_reason.py#L19

Use user.active = False https://github.com/Arpit-SerpentCS/OdooEduERP/blob/11.0_school/school/wizard/teriminate_reason.py#L29

murtuzasaleh commented 6 years ago

@Arpit-SerpentCS No need of data if noupdate="0". https://github.com/Arpit-SerpentCS/OdooEduERP/blob/11.0_school/school/security/school_security.xml#L4

murtuzasaleh commented 6 years ago

@Arpit-SerpentCS Please add domain inside the search view. https://github.com/Arpit-SerpentCS/OdooEduERP/blob/11.0_school/school/models/school.py#L224

murtuzasaleh commented 6 years ago

@Arpit-SerpentCS Both are char fields, IMO no need for str function. https://github.com/Arpit-SerpentCS/OdooEduERP/blob/11.0_school/school/models/teacher.py#L146

Arpit-SerpentCS commented 6 years ago

@murtuzasaleh Made improvements and tested the code according to comments.

cc @JayVora-SerpentCS