Skip to content
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

[l10n_xx] auto_apply on fiscal positions for all localizations #32126

Open
wants to merge 11 commits into
base: master
from

Conversation

Projects
None yet
4 participants
@hbh-odoo
Copy link
Contributor

hbh-odoo commented Mar 26, 2019

Description of the issue/feature this PR addresses:

Task: https://www.odoo.com/web?#id=1952975&action=333&active_id=131&model=project.task&view_type=form&menu_id=4720
Pad: https://pad.odoo.com/p/r.a89da88f7ef0f7e08039343c9fb8bfee

Current behavior before PR:

Desired behavior after PR is merged:

--
I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr

@robodoo robodoo added the CI 🤖 label Mar 26, 2019

@hbh-odoo hbh-odoo force-pushed the odoo-dev:master-auto-apply-fiscal-position-hbh branch from 376f2f7 to 75a7802 Mar 26, 2019

@C3POdoo C3POdoo added the RD label Mar 26, 2019

fiscal_position_tax_inclusive_template,l10n_jp1,内税
fiscal_position_tax_exclusive_template,l10n_jp1,外税
fiscal_position_tax_exempt_template,l10n_jp1,海外取引先
id,chart_template_id/id,name,sequence,auto_apply,country_id:id

This comment has been minimized.

Copy link
@jco-odoo

jco-odoo Apr 12, 2019

Contributor

With the inclusive and exclusive taxes, we can not know which one is preferred. So, I would leave it out, unless @yostashiro maybe you know if these 2 fiscal positions make sense or which one takes precedence to apply automatically.

@@ -7,16 +7,23 @@
<record id="fiscal_position_template_1" model="account.fiscal.position.template">
<field name="name">Kraj</field>
<field name="chart_template_id" ref="pl_chart_template"/>

This comment has been minimized.

Copy link
@jco-odoo

jco-odoo Apr 12, 2019

Contributor

no sequence here?

This comment has been minimized.

Copy link
@hbh-odoo

hbh-odoo Apr 12, 2019

Author Contributor

set the sequence '1'

"gd_fp_eu","gd_chart","Partner EU"
"gd_fp_ne","gd_chart","Partner izven EU / EU brez DŠ"
"id","chart_template_id/id","name","sequence","auto_apply","country_id:id","country_group_id:id"
"gd_fp_do","gd_chart","Domači partner",,1,base.si,

This comment has been minimized.

Copy link
@jco-odoo

jco-odoo Apr 12, 2019

Contributor

sequence?

<field name="name">Régimen Extracomunitario / Canarias, Ceuta y Melilla</field>
<field name="chart_template_id" ref="account_chart_template_common"/>
<field name="auto_apply" eval="True"/>

This comment has been minimized.

Copy link
@jco-odoo

jco-odoo Apr 12, 2019

Contributor

You can leave it like this for now. If there are partners within Canarias, ... they can put it on the partner and we will check later for a full solution.

This comment has been minimized.

Copy link
@hbh-odoo

hbh-odoo Apr 12, 2019

Author Contributor

Ok.

@hbh-odoo hbh-odoo force-pushed the odoo-dev:master-auto-apply-fiscal-position-hbh branch from 75a7802 to 8e9c06c Apr 12, 2019

@robodoo robodoo added CI 🤖 and removed CI 🤖 labels Apr 12, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.