Skip to content

Conversation

@Liinares
Copy link

No description provided.

@robodoo
Copy link

robodoo commented Jan 26, 2026

This PR targets the un-managed branch odoo-dev/odoo:19.0-l10n_es-accounting-onboarding-malb, it needs to be retargeted before it can be merged.

Copy link

@FlorianGilbert FlorianGilbert left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

May i ask you to explain in your commit message, the change, the reason why you did the change and the task id at the end of the commit message?

Guidelines

<field name="arch" type="xml">
<xpath expr="//page[@id='other_tab']//field[@name='delivery_date']" position="after">
<field name="l10n_es_is_simplified"/>
<field name="l10n_es_is_simplified" invisible="country_code != 'ES'" readonly="state != 'draft'"/>

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
<field name="l10n_es_is_simplified" invisible="country_code != 'ES'" readonly="state != 'draft'"/>
<field name="l10n_es_is_simplified" invisible="country_code != 'ES'" readonly="state == 'posted'"/>

Prefer this way, because we have other state and we only want readonly on it when it's posted!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants