-
-
Notifications
You must be signed in to change notification settings - Fork 294
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
8.0 mgmtsystem review #51
Conversation
migrations/7.0.1.0/ folder is not needed anymore. Please change icon path and check Travis. |
2480c36
to
283b454
Compare
Waiting on dependencies. |
@@ -19,7 +19,7 @@ | |||
# | |||
############################################################################## | |||
|
|||
from osv import fields, orm | |||
from openerp.osv import fields, orm |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is osv still needed?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It can probably get removed
db6087c
to
5ef7487
Compare
👍 |
53f41d0
to
fbfd8cf
Compare
Fix imports and replaced survey.responses by survey.user_input Removed vim lines [IMP] Add README.rst and data, views and models directories Remove executable flag on python files and xml Moved icon [IMP] - mgmtsystem_review: Improved view and fixed typo. [ADD] - mgmtsystem_review: Updated pot and added spanish translation. [FIX] - mgmtsystem_review: Fixed translation template. [REM] Migration files [IMP] Description, version [IMP] Add "Try me" button [IMP] Remove osv [FIX] Formatting in README.rst [FIX] import models [IMP] Migrate to new API [FIX] AttributeError: 'module' object has no attribute 'Many2Many' [FIX] Trailing comma after field definition
491b2b5
to
1a41e19
Compare
👍 |
Depends on