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

[ADD] l10n_in: add new field for set trade name in Indian companies #165185

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

zepa-odoo
Copy link
Contributor

Before this PR:

Supplier trade name set to company partner name.

After this PR:

If available, update logic to assign supplier trade name based on the company's official business name (trade name). You can use the company partner name as a fallback.
Impact:

Enhances consistency in supplier data representation. It'll ensure alignment with the company's official business name when applicable.

Task ID: 3887828

@robodoo
Copy link
Contributor

robodoo commented May 13, 2024

@C3POdoo C3POdoo requested review from a team and JulienVR and removed request for a team May 13, 2024 07:13
@C3POdoo C3POdoo added the RD research & development, internal work label May 13, 2024
Before this Commit:
Supplier trade name set to company partner name.

After this Commit:
Update logic to assign supplier trade name based on the company's official business name (trade name) if available.
Use the company partner name as a fallback.
Impact:

Enhances consistency in supplier data representation.
It'll make sure that alignment with the company's official business name when applicable.

Task ID: 3887828
@@ -28,6 +28,7 @@ class ResPartner(models.Model):
)

display_pan_warning = fields.Boolean(string="Display pan warning", compute="_compute_display_pan_warning")
l10n_in_trade_name = fields.Char(string="Trade name")
Copy link
Contributor

Choose a reason for hiding this comment

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

@ajjo-odoo @zepa-odoo It is making it cumbersome for the user to also have to fill that in on every partner/contact.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
RD research & development, internal work
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants