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
g[ADD] l10n_mn: Add Mongolia Accounting #31563
Conversation
@@ -0,0 +1,2 @@ | |||
id,property_account_receivable_id/id,property_account_payable_id/id,property_account_expense_categ_id/id,property_account_income_categ_id/id,income_currency_exchange_account_id/id,expense_currency_exchange_account_id/id | |||
l10n_mn.mn_chart_1,l10n_mn.account_template_1201_0201,l10n_mn.account_template_3101_0201,l10n_mn.account_template_7002_1101,l10n_mn.account_template_5101_0201,l10n_mn.account_template_5301_0201,l10n_mn.account_template_5302_0201 |
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.
Hello @jco-odoo
Plz, Verify this default account template
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.
This is clearer to me in the xml. Did you find this in the document? (I can ask)
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.
No, I can't Find this in Documents.
But as per knowledge, I do this.
So need to Verify 😃
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.
Can you remove this csv and put it in the xml? So you just need default account payable/receivable?
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 comments to check we are going in the right direction.
tax_group_no_vat_0,No VAT | ||
tax_group_0,Zero VAT | ||
tax_group_10,VAT 10% | ||
tax_group_export_vat_0,Exporting VAT 0% |
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.
Should not we just have one tax group with 0% instead of 3?
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.
Yes, We Can Do it If tax report no need separate value for this three 0% tax.
l10n_mn.tax_purchase_vat_10,Purchase VAT,10.00%,purchase,percent,10,l10n_mn.mn_chart_1,1,l10n_mn.tax_group_10 | ||
l10n_mn.tax_sale_no_vat_0,No VAT,0.00%,sale,percent,0,l10n_mn.mn_chart_1,2,l10n_mn.tax_group_no_vat_0 | ||
l10n_mn.tax_sale_zero_vat_0,Zero VAT,0.00%,sale,percent,0,l10n_mn.mn_chart_1,3,l10n_mn.tax_group_0 | ||
l10n_mn.tax_sale_export_vat_0,Exporting VAT,0.00%,sale,percent,0,l10n_mn.mn_chart_1,4,l10n_mn.tax_group_export_vat_0 |
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.
There is a separate csv with all the taxes where almost all the work is done. You should take that one.
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.
ok
<field name="name">Mongolia</field> | ||
<field name="bank_account_code_prefix">11</field> | ||
<field name="cash_account_code_prefix">10</field> | ||
<field name="transfer_account_code_prefix">16</field> |
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.
Check comments on task
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.
Ok Done
<field name="transfer_account_code_prefix">16</field> | ||
<field name="code_digits">8</field> | ||
<field name="currency_id" ref="base.MNT"/> | ||
<field name="spoken_languages" eval="'lang_mn'"/> |
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.
We don't want multilang, so I don't know what this spoken_languages is doing here
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.
Right, I remove it.
68f7a37
to
8a26c0e
Compare
8a26c0e
to
b57e77f
Compare
Hello @jco-odoo |
22e232b
to
4fc1025
Compare
4fc1025
to
c6fd624
Compare
32901a4
to
40bdd64
Compare
40bdd64
to
cf52e0f
Compare
52356e5
to
e8c775a
Compare
e8c775a
to
5a7ab30
Compare
5a7ab30
to
74e4333
Compare
Replace on #32830 |
Task: https://www.odoo.com/web#id=1946238&action=327&model=project.task&view_type=form&menu_id=4720
Pad: https://pad.odoo.com/p/r.4dae8eac38283f5748a45d5f983adb9e
--
I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr