-
-
Notifications
You must be signed in to change notification settings - Fork 695
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
[10.0][MIG] stock_valuation_account_manual_adjustment #371
[10.0][MIG] stock_valuation_account_manual_adjustment #371
Conversation
6616e38
to
7fb6feb
Compare
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.
Some remarks.
stock_valuation_account_manual_adjustment/models/account_move.py
Outdated
Show resolved
Hide resolved
stock_valuation_account_manual_adjustment/models/stock_valuation_account_manual_adjustment.py
Outdated
Show resolved
Hide resolved
..._valuation_account_manual_adjustment/tests/test_stock_valuation_account_manual_adjustment.py
Outdated
Show resolved
Hide resolved
stock_valuation_account_manual_adjustment/wizards/mass_create.py
Outdated
Show resolved
Hide resolved
stock_valuation_account_manual_adjustment/wizards/mass_create.py
Outdated
Show resolved
Hide resolved
@lreficent Any news on this? |
@rousseldenis Sorry I will not work on this at the moment, I will eventually do it but it could be not in the short term... |
59e97dd
to
e58c0e1
Compare
@rousseldenis Sorry, I completely forgot about this 😞 I'm now fixing, see my comments in your review |
6e6c1be
to
68a4815
Compare
@rousseldenis Could you check my latest comments? |
@lreficent Can you squash in order to merge this ? |
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.
Code review
[ADD] stock_valuation_account_manual_adjustment
efba1fc
to
9c8ad0a
Compare
Migration to v10.