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] l10n_br_stock_account #706
[10.0][MIG] l10n_br_stock_account #706
Conversation
ab0598b
to
2982291
Compare
OCA/account-fiscal-rule#90 has been merged. This should be rebased to avoid the postgres error with .travis.yml. |
2982291
to
3e50661
Compare
I just did the rebase on 10.0, let's see the tests now... |
I think the pot translation file is missing |
3e50661
to
07b0282
Compare
'depends': [ | ||
'l10n_br_account_product', | ||
'l10n_br_stock', | ||
'account_fiscal_position_rule_stock', | ||
'stock_account', | ||
'sale_stock', |
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.
Esta dependência deveria estar no módulo l10n_br_sale_stock. https://github.com/OCA/l10n-brazil/blob/10.0/l10n_br_sale_stock/__manifest__.py#L14
</field> | ||
</record> | ||
|
||
<!--<record model="ir.ui.view" id="l10n_br_stock_account_procurement_rule_inherit_form"> |
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.
Remover estes comentários
comment = '' | ||
if picking.fiscal_position_id.inv_copy_note: | ||
comment += picking.fiscal_position_id.note or '' | ||
if picking.sale_id and picking.sale_id.copy_note: |
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.
Esta parte deste código deveria estar no módulo l10n_br_sale_stock
<field name="fiscal_category_id" ref="l10n_br_account_product.fc_86d8c770fc2fb9d9fa242a3bdddd507a"/> | ||
</record> | ||
|
||
<!-- Picking Test - 2 |
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.
Remover comentários
284c091
to
56e63f8
Compare
Kudos @mbcosta! |
Thanks @renatonlima for review |
…ed in module sale_stock.
…n module l10n_br_sale_stock.
c463a1c
to
2a80e77
Compare
/ocabot merge |
On my way to merge this fine PR! |
Congratulations, your PR was merged at 1b62878. Thanks a lot for contributing to OCA. ❤️ |
Hi @gabrielcardoso21 , thanks by review |
…achment [REM] Non necessary code
Replace #640 to use a common repository.
Depends OCA/account-fiscal-rule#90
WIP - Procurement
cc @renatonlima @rvalyi @mileo