Skip to content

Conversation

@vib-adhoc
Copy link
Contributor

No description provided.

Copilot AI review requested due to automatic review settings November 26, 2025 16:39
@roboadhoc
Copy link
Contributor

Pull request status dashboard

@vib-adhoc vib-adhoc merged commit 2bfd049 into 19.0 Nov 26, 2025
3 checks passed
@vib-adhoc vib-adhoc deleted the 19.0-vib-4 branch November 26, 2025 16:39
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds a new otro_test field to the res.partner model extension and updates the translation file with corresponding entries. The changes maintain consistency with the existing translation infrastructure by automatically generating proper translation entries for the new field.

  • Adds otro_test field to the partner model
  • Updates POT translation file creation and revision dates
  • Adds translation entries for the new field across res_partner and res_users models

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
partner_sales_abc/models/res_partner.py Adds new otro_test Char field to the partner model
partner_sales_abc/i18n/partner_sales_abc.pot Updates translation timestamps and adds translation entries for the new field

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

_inherit = "res.partner"
abc_sales_amount = fields.Char("ABC Sales amount asd")
test_field = fields.Char()
otro_test = fields.Char()
Copy link

Copilot AI Nov 26, 2025

Choose a reason for hiding this comment

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

The new field otro_test is missing a string description parameter. Consider adding a descriptive label like fields.Char("Otro Test") to provide a user-friendly name in the UI, consistent with how abc_sales_amount is defined on line 11.

Suggested change
otro_test = fields.Char()
otro_test = fields.Char("Otro Test")

Copilot uses AI. Check for mistakes.
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.

3 participants