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
[REL] stock_inventory_require_lot to V11. #82
Conversation
ValentinoDefelice
commented
Jun 28, 2018
- [FIX] Separe models and views in diferent files.
- [FIX] Update Manifest and README
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.
Hi Valen!
Thanks for this PR.
I leave some comments below.
:target: https://runbot.adhoc.com.ar/ | ||
|
||
.. repo_id is available in https://github.com/OCA/maintainer-tools/blob/master/tools/repos_with_ids.txt | ||
.. branch is "9.0" for example | ||
|
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.
Here you need the part of the configuration, installation and usage.
_inherit = 'stock.change.product.qty' | ||
|
||
tracking = fields.Selection( | ||
related='product_id.tracking', readonly=True |
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.
Valen, here it would be much better this way:
tracking = fields.Selection(
related='product_id.tracking',
readonly=True,
)
|
||
_inherit = 'stock.inventory.line' | ||
|
||
tracking = fields.Selection( |
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.
The same thing that I put in the previous comment.
* [FIX] Separe models and views in diferent files. * [FIX] Update Manifest and README
51eb7a6
to
3a17140
Compare
Codecov Report
@@ Coverage Diff @@
## 11.0 #82 +/- ##
==========================================
+ Coverage 46.19% 48.16% +1.97%
==========================================
Files 9 11 +2
Lines 210 218 +8
==========================================
+ Hits 97 105 +8
Misses 113 113
Continue to review full report at Codecov.
|