Skip to content

Conversation

@augusto-weiss
Copy link
Contributor

No description provided.

@augusto-weiss augusto-weiss marked this pull request as ready for review November 20, 2025 18:58
Copilot AI review requested due to automatic review settings November 20, 2025 18:58
@augusto-weiss augusto-weiss mentioned this pull request Nov 20, 2025
19 tasks
Copilot finished reviewing on behalf of augusto-weiss November 20, 2025 19:01
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 migrates the base_user_show_email module from version 18.0 to 19.0. The module enhances Odoo's user interface by clearly separating the login and email fields, which are tightly coupled in recent Odoo versions.

Key changes:

  • Updated module version to 19.0.1.0.0 in the manifest
  • Migrated view definitions (tree view and login layout)
  • Included translation files and documentation

Reviewed Changes

Copilot reviewed 10 out of 12 changed files in this pull request and generated 8 comments.

Show a summary per file
File Description
base_user_show_email/manifest.py Updated version to 19.0.1.0.0 and defined module metadata
base_user_show_email/init.py Empty init file for module structure
base_user_show_email/views/res_users_view.xml Extends res.users tree view to display email field next to login
base_user_show_email/views/login_layout.xml Customizes login template to display "Login" label instead of "Email"
base_user_show_email/readme/DESCRIPTION.md Source file for module description (contains grammar error and mentions missing form view)
base_user_show_email/readme/CONTRIBUTORS.md Lists module contributors
base_user_show_email/README.rst Auto-generated documentation (has outdated 18.0 version references)
base_user_show_email/static/description/index.html Auto-generated HTML documentation (has outdated 18.0 version references)
base_user_show_email/static/description/icon.png Module icon (binary file)
base_user_show_email/i18n/base_user_show_email.pot Translation template (has outdated 18.0 version and references missing form view)
base_user_show_email/i18n/it.po Italian translation (has outdated 16.0 version and references missing form view)
base_user_show_email/pyproject.toml Build system configuration using whool

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

mission is to support the collaborative development of Odoo features and
promote its widespread use.

This module is part of the `OCA/server-auth <https://github.com/OCA/server-auth/tree/18.0/base_user_show_email>`_ project on GitHub.
Copy link

Copilot AI Nov 20, 2025

Choose a reason for hiding this comment

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

The version references in this file still point to 18.0 but should be updated to 19.0 for this migration. The following lines need updating:

  • Line 20: tree/18.0/base_user_show_email → tree/19.0/base_user_show_email
  • Line 23: server-auth-18-0 → server-auth-19-0 (twice)
  • Line 26: target_branch=18.0 → target_branch=19.0
  • Line 59: version:%2018.0 → version:%2019.0
  • Line 89: tree/18.0/base_user_show_email → tree/19.0/base_user_show_email
Suggested change
This module is part of the `OCA/server-auth <https://github.com/OCA/server-auth/tree/18.0/base_user_show_email>`_ project on GitHub.
This module is part of the `OCA/server-auth <https://github.com/OCA/server-auth/tree/19.0/base_user_show_email>`_ project on GitHub.

Copilot uses AI. Check for mistakes.
<p>OCA, or the Odoo Community Association, is a nonprofit organization whose
mission is to support the collaborative development of Odoo features and
promote its widespread use.</p>
<p>This module is part of the <a class="reference external" href="https://github.com/OCA/server-auth/tree/18.0/base_user_show_email">OCA/server-auth</a> project on GitHub.</p>
Copy link

Copilot AI Nov 20, 2025

Choose a reason for hiding this comment

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

The version references in this file still point to 18.0 but should be updated to 19.0 for this migration. The following need updating:

  • Line 372: tree/18.0/base_user_show_email → tree/19.0/base_user_show_email, server-auth-18-0 → server-auth-19-0 (twice), target_branch=18.0 → target_branch=19.0
  • Line 410: version:%2018.0 → version:%2019.0
  • Line 436: tree/18.0/base_user_show_email → tree/19.0/base_user_show_email
Suggested change
<p>This module is part of the <a class="reference external" href="https://github.com/OCA/server-auth/tree/18.0/base_user_show_email">OCA/server-auth</a> project on GitHub.</p>
<p>This module is part of the <a class="reference external" href="https://github.com/OCA/server-auth/tree/19.0/base_user_show_email">OCA/server-auth</a> project on GitHub.</p>

Copilot uses AI. Check for mistakes.
#
msgid ""
msgstr ""
"Project-Id-Version: Odoo Server 16.0\n"
Copy link

Copilot AI Nov 20, 2025

Choose a reason for hiding this comment

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

The Project-Id-Version references Odoo Server 16.0, but this is a migration to 19.0. While translation files may be carried over from previous versions, for consistency and clarity, this should be updated to "Odoo Server 19.0".

Suggested change
"Project-Id-Version: Odoo Server 16.0\n"
"Project-Id-Version: Odoo Server 19.0\n"

Copilot uses AI. Check for mistakes.
Comment on lines +11 to +14
- In the res.users form view:
- Display a "Login" label instead of "Email" on the login field
- Show the currently invisible email field and its label

Copy link

Copilot AI Nov 20, 2025

Choose a reason for hiding this comment

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

The DESCRIPTION.md mentions changes to "the res.users form view" (lines 11-13), but the README.rst (which is generated from the description) omits this section and only mentions the tree view (line 41) and login layout (line 45). Additionally, the translation files reference base_user_show_email.user_email_form view, but no form view is included in the views directory. Either:

  1. A form view file is missing from this migration, or
  2. The DESCRIPTION.md is outdated and should be updated to remove the form view references

This inconsistency should be resolved to ensure the module description matches the actual implementation.

Suggested change
- In the res.users form view:
- Display a "Login" label instead of "Email" on the login field
- Show the currently invisible email field and its label

Copilot uses AI. Check for mistakes.
field (the field is displayed with an "Email" label, and it will
populate the email of the user if it contains an "@" character).

Yet for the users who uses a distinct login and email, this is
Copy link

Copilot AI Nov 20, 2025

Choose a reason for hiding this comment

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

Grammar error: "for the users who uses" should be "for the users who use" (plural subject requires plural verb form).

Suggested change
Yet for the users who uses a distinct login and email, this is
Yet for the users who use a distinct login and email, this is

Copilot uses AI. Check for mistakes.
#
msgid ""
msgstr ""
"Project-Id-Version: Odoo Server 18.0\n"
Copy link

Copilot AI Nov 20, 2025

Choose a reason for hiding this comment

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

The Project-Id-Version references "Odoo Server 18.0", but this is a migration to 19.0. This should be updated to "Odoo Server 19.0" for consistency.

Suggested change
"Project-Id-Version: Odoo Server 18.0\n"
"Project-Id-Version: Odoo Server 19.0\n"

Copilot uses AI. Check for mistakes.
Comment on lines +17 to +23
#: model_terms:ir.ui.view,arch_db:base_user_show_email.user_email_form
msgid "Email Address"
msgstr ""

#. module: base_user_show_email
#: model_terms:ir.ui.view,arch_db:base_user_show_email.login
#: model_terms:ir.ui.view,arch_db:base_user_show_email.user_email_form
Copy link

Copilot AI Nov 20, 2025

Choose a reason for hiding this comment

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

The translation files reference a view base_user_show_email.user_email_form (lines 17, 23), but this view does not exist in the module's XML files. The module's DESCRIPTION.md mentions form view modifications, but only tree view (res_users_view.xml) and login layout (login_layout.xml) files are included. Either:

  1. The form view file is missing from this migration and needs to be added, or
  2. These translation entries are outdated and should be removed

This should be investigated and resolved to ensure the module works correctly.

Suggested change
#: model_terms:ir.ui.view,arch_db:base_user_show_email.user_email_form
msgid "Email Address"
msgstr ""
#. module: base_user_show_email
#: model_terms:ir.ui.view,arch_db:base_user_show_email.login
#: model_terms:ir.ui.view,arch_db:base_user_show_email.user_email_form
#: model_terms:ir.ui.view,arch_db:base_user_show_email.login

Copilot uses AI. Check for mistakes.
Comment on lines +20 to +26
#: model_terms:ir.ui.view,arch_db:base_user_show_email.user_email_form
msgid "Email Address"
msgstr "Indirizzo e-mail"

#. module: base_user_show_email
#: model_terms:ir.ui.view,arch_db:base_user_show_email.login
#: model_terms:ir.ui.view,arch_db:base_user_show_email.user_email_form
Copy link

Copilot AI Nov 20, 2025

Choose a reason for hiding this comment

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

The translation file references a view base_user_show_email.user_email_form (lines 20, 23, 26, 27), but this view does not exist in the module's XML files. Either this form view is missing from the migration, or these translation entries are outdated and should be removed. This should be synchronized with the actual module implementation.

Suggested change
#: model_terms:ir.ui.view,arch_db:base_user_show_email.user_email_form
msgid "Email Address"
msgstr "Indirizzo e-mail"
#. module: base_user_show_email
#: model_terms:ir.ui.view,arch_db:base_user_show_email.login
#: model_terms:ir.ui.view,arch_db:base_user_show_email.user_email_form
#. module: base_user_show_email
#: model_terms:ir.ui.view,arch_db:base_user_show_email.login

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.

7 participants