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

[16.0][MIG] partner_noncommercial: Migration to 16.0 #1898

Open
wants to merge 23 commits into
base: 16.0
Choose a base branch
from

Conversation

ntsirintanis
Copy link
Contributor

No description provided.

NL66278 and others added 21 commits November 26, 2024 14:16
* [ENH] Migration of the 7.0 module partner_noncommercial.

This used to be part of therp-oe-staging modules, but seems
to be usefull for more organisations that are primarily or completely
non-commercial. The organisations might be put of by the commercial
lingo that is shown by default.

* [FIX] Add name / mail to contributors.
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: partner-contact-11.0/partner-contact-11.0-partner_noncommercial
Translate-URL: https://translation.odoo-community.org/projects/partner-contact-11-0/partner-contact-11-0-partner_noncommercial/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: partner-contact-12.0/partner-contact-12.0-partner_noncommercial
Translate-URL: https://translation.odoo-community.org/projects/partner-contact-12-0/partner-contact-12-0-partner_noncommercial/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: partner-contact-12.0/partner-contact-12.0-partner_noncommercial
Translate-URL: https://translation.odoo-community.org/projects/partner-contact-12-0/partner-contact-12-0-partner_noncommercial/
- Remove All Partners action and menu entry as this duplicates default contact view;
- Add some commonly used groupings and filters.
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: partner-contact-12.0/partner-contact-12.0-partner_noncommercial
Translate-URL: https://translation.odoo-community.org/projects/partner-contact-12-0/partner-contact-12-0-partner_noncommercial/
@@ -0,0 +1,3 @@
To configure this module, you need to:

#. This module requires no special configuration.
Copy link
Contributor

Choose a reason for hiding this comment

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

I think this file can be removed.

<filter
name="group_organisation"
string="Organisation"
context="{'group_by': 'commercial_partner_id'}"

Choose a reason for hiding this comment

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

There is already an Odoo default grouping by Company:

<filter name="group_company" string="Company" context="{'group_by': 'parent_id'}"/>

Wouldn't it be better to just change the string of that one?

<attribute name="string">Organisations</attribute>
</filter>

<!-- Add extra filters. -->

Choose a reason for hiding this comment

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

Odoo also has a filter where you can type a company name and then search for it:

<field name="parent_id" domain="[('is_company', '=', True)]" operator="child_of"/>

Maybe we should also rename that one to Organisation.

<!-- Add extra groupings. -->
<!-- Change string of Company group to Organisations. -->
<filter name="group_company" position="attributes">
<attribute name="string">Organisations</attribute>

Choose a reason for hiding this comment

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

singular form seems more appropriate here

@@ -7,6 +7,11 @@
<field name="mode">extension</field>
<field type="xml" name="arch">

<!-- Change string of parent_id field to Organisations. -->
<field name="parent_id" position="attributes">
<attribute name="string">Organisations</attribute>

Choose a reason for hiding this comment

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

and here as well

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.

8 participants