Skip to content

[FIX] saudi_arabia: add warning about company name length#16275

Closed
fw-bot wants to merge 1 commit intomasterfrom
master-17.0-enforce-company-name-length-zatca-viso-495913-fw
Closed

[FIX] saudi_arabia: add warning about company name length#16275
fw-bot wants to merge 1 commit intomasterfrom
master-17.0-enforce-company-name-length-zatca-viso-495913-fw

Conversation

@fw-bot
Copy link
Copy Markdown
Collaborator

@fw-bot fw-bot commented Feb 3, 2026

We have had lots of tickets recently about failed ZATCA onboarding. One of the most common reasons is that the company or branch name is too long compared to the strict limits imposed by zatca.

As for the limits, they are technical, imposed by the QR code generated for every invoice.

The limit itself is 127 bytes, which depending on the exact arabic characters used can range from 42 to 63 arabic characters. 63 is in the best-case scenario where we can encode all characters on 2 bytes.
42 characters is the worst-case scenario where special characters requiring 3 bytes are used.

In an effort to keep our documentation concise and clear, we chose to stick to the best-case scenario of 63 characters, assuming that would be the most common scenario.
Pull requests have been made in the main codebase to make the error messages clearer upon failed onboarding, which should cover the "42 characters edge cases".

Forward-Port-Of: #16218
Forward-Port-Of: #16196

We have had lots of tickets recently about failed ZATCA onboarding.
One of the most common reasons is that the company or branch name
is too long compared to the strict limits imposed by zatca.

As for the limits, they are technical, imposed by the QR code
generated for every invoice.

The limit itself is 127 bytes, which depending on the exact arabic
characters used can range from 42 to 63 arabic characters.
63 is in the best-case scenario where we can encode all characters
on 2 bytes.
42 characters is the worst-case scenario where special characters
requiring 3 bytes are used.

In an effort to keep our documentation concise and clear, we chose
to stick to the best-case scenario of 63 characters, assuming that
would be the most common scenario.
Pull requests have been made in the main codebase to make the error
messages clearer upon failed onboarding, which should cover the
"42 characters edge cases".

X-original-commit: 9fd0acc
@robodoo
Copy link
Copy Markdown
Collaborator

robodoo commented Feb 3, 2026

Pull request status dashboard

@fw-bot
Copy link
Copy Markdown
Collaborator Author

fw-bot commented Feb 3, 2026

@viso-odoo @auva-odoo this PR targets master and is the last of the forward-port chain containing:

To merge the full chain, use

@robodoo r+

More info at https://github.com/odoo/odoo/wiki/Mergebot#forward-port

@viso-odoo
Copy link
Copy Markdown
Contributor

robodoo r+

robodoo pushed a commit that referenced this pull request Feb 3, 2026
We have had lots of tickets recently about failed ZATCA onboarding.
One of the most common reasons is that the company or branch name
is too long compared to the strict limits imposed by zatca.

As for the limits, they are technical, imposed by the QR code
generated for every invoice.

The limit itself is 127 bytes, which depending on the exact arabic
characters used can range from 42 to 63 arabic characters.
63 is in the best-case scenario where we can encode all characters
on 2 bytes.
42 characters is the worst-case scenario where special characters
requiring 3 bytes are used.

In an effort to keep our documentation concise and clear, we chose
to stick to the best-case scenario of 63 characters, assuming that
would be the most common scenario.
Pull requests have been made in the main codebase to make the error
messages clearer upon failed onboarding, which should cover the
"42 characters edge cases".

closes #16275

X-original-commit: 9fd0acc
Signed-off-by: Audrey Vandromme (auva) <auva@odoo.com>
Signed-off-by: Victorien Sonneville (viso) <viso@odoo.com>
@robodoo robodoo closed this Feb 3, 2026
@robodoo robodoo added the 19.2 label Feb 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants