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.

Sign up for GitHub

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

Jump to bottom

[18.0][MIG] partner_multi_relation: Migration to 18.0 #1933

Open
wants to merge 44 commits into
base: 18.0
Choose a base branch
from
Open

[18.0][MIG] partner_multi_relation: Migration to 18.0 #1933

wants to merge 44 commits into from

Conversation

BT-dmoreno
Copy link

No description provided.

NL66278 and others added 30 commits November 12, 2024 15:29
@NL66278 @BT-mchechlacz
[MIG] partner_multi_relation: Migration to 10.0
a9f06ec
@gurneyalex @BT-mchechlacz
[FIX] partner_multi_relation: Several fixes:
0570219 
* API of _auto_init
* Menu and groups
  stuff has moved from base to sales_team in odoo 10
@chienandalu @BT-mchechlacz
[IMP] Menu order and v10 deprecations
fa88324
@pedrobaeza @BT-mchechlacz
[FIX] partner_multi_relation: Fix errors on searches
2686fb9 
* Fix error on search when leaf is (1, '=', 1)
* Another comparison fix with search arguments
@NL66278 @BT-mchechlacz
[IMP] Make partner_multi_relation more extendable.
2d840f5
@NL66278 @BT-mchechlacz
[FIX] partner_multi_relation. Drop is_inverse from vals on creation o…
a63805e 
…f relation.
@NL66278 @BT-mchechlacz
[FIX] partner_multi_relation. Wrong self instead of this.
960457c
@ddufresne @BT-mchechlacz
[MIG] partner_multi_relation: Migration to 11.0
439a780 
* Update README
* Python 2to3
* Move relation menus items under contacts
* Fix date comparison
* Fix cache invalidate on relation write
* Bump module version
* Remove test_relation_type_unlink_dberror

  This test is not required because it tests a standard behavior of SQL.
  If a foreign key column is not nullable, then the foreign row can not be deleted.

* Remove utf8 encoding comment
* Add missing dependency for partner_multi_relation
* Add missing active_test in view action
@NL66278 @BT-mchechlacz
[FIX] partner_multi_relation. Extendable but threadsafe.
8f12d10
@ddufresne @BT-mchechlacz
[MIG] partner_multi_relation: Continue migration to 11.0
e1a10a5 
* Bug Fix
* Enable additional columns on res_partner_relation_all.

  Enable to add additional columns from res_partner_relation in the sql view of res_partner_relation_all.

* Only require inverse_name when relation type is not symmetric
* Remove deprecated @api.one and replace openerp with odoo
* Add website to manifest
* Prevent disabling allow_self if reflexive relations exist
* Enable deleting/ending invalid relations after deactivating reflexivity.
* Handle case of invalid future reflexive relations
@BT-mchechlacz
Migrate partner_multi_relation to version 12.0
d07868b
@Galoperd @BT-mchechlacz
Translated using Weblate (Spanish)
a965750 
Currently translated at 86.6% (84 of 97 strings)

Translation: partner-contact-12.0/partner-contact-12.0-partner_multi_relation
Translate-URL: https://translation.odoo-community.org/projects/partner-contact-12-0/partner-contact-12-0-partner_multi_relation/es/
@osi-scampbell @BT-mchechlacz
[FIX] Context Errors With Active ID
01442bb 
[IMP] Flake8
@osi-scampbell @BT-mchechlacz
[IMP] Formatting
d979b69
@skeller1 @BT-mchechlacz
[FIX] remove import statements for tests
53b0b8a
@osi-scampbell @BT-mchechlacz
[IMP] Flake8
a663e78
@skeller1 @BT-mchechlacz
[FIX] partner_multi_relation: fix unlinking of record sets with same …
4eb375d 
…res_id
@patrickrwilson @BT-mchechlacz
[FIX] partner_multi_relation (Smart Button Error)
3e10a65 
Fix for error when opening a contact and clicking the smart button.

Error Information:
  File "/opt/odoo/v12/src/partner-contact/partner_multi_relation/models/res_partner.py", line 187, in action_view_relations
    action['context'].\
AttributeError: 'str' object has no attribute 'update'
@patrickrwilson @BT-mchechlacz
[UPD] Attempt at manipulating context
8b3f32d
@patrickrwilson @BT-mchechlacz
[FIX] Contact ID Error
a008c64
@SicurSam @BT-mchechlacz
Translated using Weblate (Italian)
92d147a 
Currently translated at 46.4% (45 of 97 strings)

Translation: partner-contact-12.0/partner-contact-12.0-partner_multi_relation
Translate-URL: https://translation.odoo-community.org/projects/partner-contact-12-0/partner-contact-12-0-partner_multi_relation/it/
@NL66278 @BT-mchechlacz
[MIG] partner_multi_relation. Black, isort, pretty...
6fe2139
@NL66278 @BT-mchechlacz
[MIG] partner_multi_relation. Actual 13.0 migration.
aa2ded0
@BT-mchechlacz
[FIX] Fixing problems with migration of computed fields with search f…
f19b0f8 
…unction and no computed function defined
@ibuioli @BT-mchechlacz
Translated using Weblate (Spanish (Argentina))
5ecf7c3 
Currently translated at 100.0% (97 of 97 strings)

Translation: partner-contact-13.0/partner-contact-13.0-partner_multi_relation
Translate-URL: https://translation.odoo-community.org/projects/partner-contact-13-0/partner-contact-13-0-partner_multi_relation/es_AR/
@rven @BT-mchechlacz
[IMP] partner_multi_relation: black, isort
49f8e1f
@rven @BT-mchechlacz
[MIG] partner_multi_relation: Migration to 14.0
60dc61e
@BT-mchechlacz
[FIX] partner_multi_relation: Avoid access error on Relations button
ab4c8dc 
In Odoo v14, ir.actions.act_window have only access rights for admin while a user had read access in previous versions. Odoo provides now a function (_for_xml_id) that returns the action read with sudo.
@pedrocs-exo @BT-mchechlacz
Translated using Weblate (Portuguese)
0eb0084 
Currently translated at 100.0% (97 of 97 strings)

Translation: partner-contact-14.0/partner-contact-14.0-partner_multi_relation
Translate-URL: https://translation.odoo-community.org/projects/partner-contact-14-0/partner-contact-14-0-partner_multi_relation/pt/
@BT-mchechlacz
[MIG] partner_multi_relation: Migration to 15.0
a59fcee 
[UPD] Update partner_multi_relation.pot
sysadminmatmoz and others added 10 commits November 12, 2024 15:29
@sysadminmatmoz @BT-mchechlacz
Translated using Weblate (Slovenian)
ebad457 
Currently translated at 85.4% (82 of 96 strings)

Translation: partner-contact-16.0/partner-contact-16.0-partner_multi_relation
Translate-URL: https://translation.odoo-community.org/projects/partner-contact-16-0/partner-contact-16-0-partner_multi_relation/sl/
@Ivorra78 @BT-mchechlacz
Translated using Weblate (Spanish)
b9526e6 
Currently translated at 100.0% (96 of 96 strings)

Translation: partner-contact-16.0/partner-contact-16.0-partner_multi_relation
Translate-URL: https://translation.odoo-community.org/projects/partner-contact-16-0/partner-contact-16-0-partner_multi_relation/es/
@mymage @BT-mchechlacz
Translated using Weblate (Italian)
1ea0c81 
Currently translated at 100.0% (96 of 96 strings)

Translation: partner-contact-16.0/partner-contact-16.0-partner_multi_relation
Translate-URL: https://translation.odoo-community.org/projects/partner-contact-16-0/partner-contact-16-0-partner_multi_relation/it/
@BT-mchechlacz
[IMP] partner_multi_relation: pre-commit stuff
5cf21cd
@BT-mchechlacz
[MIG] partner_multi_relation: Migration to 17.0
472e082
@BT-mchechlacz
[UPD] Update partner_multi_relation.pot
9371a27
@weblate @BT-mchechlacz
Update translation files
c15e12d 
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: partner-contact-17.0/partner-contact-17.0-partner_multi_relation
Translate-URL: https://translation.odoo-community.org/projects/partner-contact-17-0/partner-contact-17-0-partner_multi_relation/
@jakobkrabbe @BT-mchechlacz
Translated using Weblate (Swedish)
aa4dbaf 
Currently translated at 100.0% (95 of 95 strings)

Translation: partner-contact-17.0/partner-contact-17.0-partner_multi_relation
Translate-URL: https://translation.odoo-community.org/projects/partner-contact-17-0/partner-contact-17-0-partner_multi_relation/sv/
@jdidderen-nsi @BT-mchechlacz
[IMP] partner_contact_role: pre-commit execution
6d180bd
@BT-mchechlacz
[IMP] partner_multi_relation: pre-commit auto fixes
a6b2d03
@BT-dmoreno BT-dmoreno marked this pull request as draft December 16, 2024 09:39
@BT-dmoreno BT-dmoreno force-pushed the 18.0-mig-partner_multi_relation branch from 5a3c6f7 to 6dc23cf Compare December 17, 2024 06:59
@BT-dmoreno BT-dmoreno marked this pull request as ready for review December 17, 2024 07:00
@BT-dmoreno BT-dmoreno force-pushed the 18.0-mig-partner_multi_relation branch from 2475e0c to b33bfc9 Compare December 17, 2024 10:52
@BT-dmoreno
Copy link
Author

Hi @rousseldenis please could you cancel the other open PR here, targeting this same module? This new PR includes all the previous changes plus the core review changes requested by you there. I'll ask for new reviews for this PR, but if you can also take a look would be great.

The reason for the new PR is that the previous one was done from an account that is not going to be used anymore.

Also, please could you assign me the migration of this module in the issue for migrations?

@BT-dmoreno BT-dmoreno mentioned this pull request Dec 17, 2024
Closed
BT-mlopez
BT-mlopez approved these changes Dec 19, 2024
View reviewed changes
Copy link

@BT-mlopez BT-mlopez left a comment

Choose a reason for hiding this comment

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

LGTM

@rousseldenis
Copy link
Contributor

/ocabot migration partner_multi_relation

@OCA-git-bot OCA-git-bot added this to the 18.0 milestone Dec 19, 2024
@OCA-git-bot
Copy link
Contributor

The migration issue (#1856) has not been updated to reference the current pull request because a previous pull request (#1895) is not closed.
Perhaps you should check that there is no duplicate work.
CC @BT-mchechlacz

@rousseldenis
Copy link
Contributor

/ocabot migration partner_multi_relation

@OCA-git-bot OCA-git-bot mentioned this pull request Dec 19, 2024
Open
46 tasks
rousseldenis
rousseldenis approved these changes Dec 19, 2024
View reviewed changes
Copy link
Contributor

@rousseldenis rousseldenis left a comment

Choose a reason for hiding this comment

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

Code review

BT-mlopez

This comment was marked as duplicate.

Sign in to view

@OCA-git-bot
Copy link
Contributor

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@BT-mlopez BT-mlopez BT-mlopez approved these changes

@rousseldenis rousseldenis rousseldenis approved these changes

Assignees
No one assigned
Labels
approved migration needs review ready to merge
Projects
None yet
Milestone
18.0
Development

Successfully merging this pull request may close these issues.