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

[MIG] autovacuum_message_attachment: Migration to 17.0 #3183

Open
wants to merge 33 commits into
base: 17.0
Choose a base branch
from

Conversation

GabbasovDinar
Copy link
Member

No description provided.

florian-dacosta and others added 10 commits February 3, 2025 13:59

Verified

This commit was signed with the committer’s verified signature.

Verified

This commit was signed with the committer’s verified signature.

Verified

This commit was signed with the committer’s verified signature.

Verified

This commit was signed with the committer’s verified signature.

Verified

This commit was signed with the committer’s verified signature.
… it is already present in other models

Verified

This commit was signed with the committer’s verified signature.

Verified

This commit was signed with the committer’s verified signature.

Verified

This commit was signed with the committer’s verified signature.

Verified

This commit was signed with the committer’s verified signature.

Verified

This commit was signed with the committer’s verified signature.
@GabbasovDinar GabbasovDinar mentioned this pull request Feb 3, 2025
35 tasks
Copy link
Contributor

@CRogos CRogos left a comment

Choose a reason for hiding this comment

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

I am not using this module so I don't know how it works, but looks like a interesting and very powerfull module. Maybe add some more test coverage?

Also some commits could be squashed. Here some examples:
image

The changes so fare LGTM.

@mike-cetmix
Copy link

@CRogos Updated the readme to make the module behaviour more clear

@CRogos
Copy link
Contributor

CRogos commented Feb 6, 2025

@Bearnard21 I apricate your work. Could you also do the squashing?

Copy link

@hoangtrann hoangtrann left a comment

Choose a reason for hiding this comment

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

Code LGTM!

@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). 🤖

@GabbasovDinar GabbasovDinar force-pushed the 17.0-t4316-autovacuum_message_attachment-mig branch from 05a85c5 to 1f5f226 Compare February 10, 2025 07:02
@GabbasovDinar
Copy link
Member Author

@CRogos Hi, I squashed some commits

@CRogos
Copy link
Contributor

CRogos commented Feb 10, 2025

There are still weblate and oca-ci commits which can be squashed.

ibuioli and others added 8 commits February 11, 2025 16:24

Verified

This commit was signed with the committer’s verified signature.
Translated using Weblate (Spanish (Argentina))

Currently translated at 100.0% (50 of 50 strings)

Translation: server-tools-14.0/server-tools-14.0-autovacuum_message_attachment
Translate-URL: https://translation.odoo-community.org/projects/server-tools-14-0/server-tools-14-0-autovacuum_message_attachment/es_AR/

Verified

This commit was signed with the committer’s verified signature.

Verified

This commit was signed with the committer’s verified signature.
Selection of attachments through inheriting model can be useful if
we want to purge attachments that are extended through another
model.

For example, shipping.label (from base_delivery_carrier_label)
inherits from ir.attachment but the attachment itself is linked to
the stock.picking model.
By using inheriting_model defined to shipping.label and setting
model_ids to stock.picking, we can safely select the attachments
linked to stock.pickings that are shipping labels and avoid deleting
any picking list or delivery note reports.

Verified

This commit was signed with the committer’s verified signature.
Currently translated at 100.0% (55 of 55 strings)

Translation: server-tools-14.0/server-tools-14.0-autovacuum_message_attachment
Translate-URL: https://translation.odoo-community.org/projects/server-tools-14-0/server-tools-14-0-autovacuum_message_attachment/es_AR/

Verified

This commit was signed with the committer’s verified signature.

Verified

This commit was signed with the committer’s verified signature.

Verified

This commit was signed with the committer’s verified signature.
But allow it only if a file pattern is set, to avoid deleting all orphan attachments are some are auto-generated by Odoo

Verified

This commit was signed with the committer’s verified signature.
mymage and others added 15 commits February 11, 2025 16:25

Verified

This commit was signed with the committer’s verified signature.
Translated using Weblate (Italian)

Currently translated at 100.0% (52 of 52 strings)

Translation: server-tools-16.0/server-tools-16.0-autovacuum_message_attachment
Translate-URL: https://translation.odoo-community.org/projects/server-tools-16-0/server-tools-16-0-autovacuum_message_attachment/it/

Verified

This commit was signed with the committer’s verified signature.
Translated using Weblate (Spanish)

Currently translated at 67.3% (35 of 52 strings)

Translation: server-tools-16.0/server-tools-16.0-autovacuum_message_attachment
Translate-URL: https://translation.odoo-community.org/projects/server-tools-16-0/server-tools-16-0-autovacuum_message_attachment/es/

Translated using Weblate (Spanish)

Currently translated at 100.0% (52 of 52 strings)

Translation: server-tools-16.0/server-tools-16.0-autovacuum_message_attachment
Translate-URL: https://translation.odoo-community.org/projects/server-tools-16-0/server-tools-16-0-autovacuum_message_attachment/es/

Verified

This commit was signed with the committer’s verified signature.

Verified

This commit was signed with the committer’s verified signature.

Verified

This commit was signed with the committer’s verified signature.

Verified

This commit was signed with the committer’s verified signature.
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: server-tools-16.0/server-tools-16.0-autovacuum_message_attachment
Translate-URL: https://translation.odoo-community.org/projects/server-tools-16-0/server-tools-16-0-autovacuum_message_attachment/

Verified

This commit was signed with the committer’s verified signature.
Currently translated at 100.0% (52 of 52 strings)

Translation: server-tools-16.0/server-tools-16.0-autovacuum_message_attachment
Translate-URL: https://translation.odoo-community.org/projects/server-tools-16-0/server-tools-16-0-autovacuum_message_attachment/es_AR/

Verified

This commit was signed with the committer’s verified signature.
Update translation files

Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: server-tools-16.0/server-tools-16.0-autovacuum_message_attachment
Translate-URL: https://translation.odoo-community.org/projects/server-tools-16-0/server-tools-16-0-autovacuum_message_attachment/

Verified

This commit was signed with the committer’s verified signature.
Currently translated at 100.0% (61 of 61 strings)

Translation: server-tools-16.0/server-tools-16.0-autovacuum_message_attachment
Translate-URL: https://translation.odoo-community.org/projects/server-tools-16-0/server-tools-16-0-autovacuum_message_attachment/it/

Verified

This commit was signed with the committer’s verified signature.
Update translation files

Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: server-tools-16.0/server-tools-16.0-autovacuum_message_attachment
Translate-URL: https://translation.odoo-community.org/projects/server-tools-16-0/server-tools-16-0-autovacuum_message_attachment/

Verified

This commit was signed with the committer’s verified signature.
Currently translated at 100.0% (61 of 61 strings)

Translation: server-tools-16.0/server-tools-16.0-autovacuum_message_attachment
Translate-URL: https://translation.odoo-community.org/projects/server-tools-16-0/server-tools-16-0-autovacuum_message_attachment/es_AR/

Verified

This commit was signed with the committer’s verified signature.
Currently translated at 100.0% (61 of 61 strings)

Translation: server-tools-16.0/server-tools-16.0-autovacuum_message_attachment
Translate-URL: https://translation.odoo-community.org/projects/server-tools-16-0/server-tools-16-0-autovacuum_message_attachment/it/

Verified

This commit was signed with the committer’s verified signature.
Update translation files

Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: server-tools-16.0/server-tools-16.0-autovacuum_message_attachment
Translate-URL: https://translation.odoo-community.org/projects/server-tools-16-0/server-tools-16-0-autovacuum_message_attachment/

[UPD] Update autovacuum_message_attachment.pot

Update translation files

Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: server-tools-16.0/server-tools-16.0-autovacuum_message_attachment
Translate-URL: https://translation.odoo-community.org/projects/server-tools-16-0/server-tools-16-0-autovacuum_message_attachment/

[UPD] Update autovacuum_message_attachment.pot

Update translation files

Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: server-tools-16.0/server-tools-16.0-autovacuum_message_attachment
Translate-URL: https://translation.odoo-community.org/projects/server-tools-16-0/server-tools-16-0-autovacuum_message_attachment/

[UPD] Update autovacuum_message_attachment.pot

Update translation files

Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: server-tools-16.0/server-tools-16.0-autovacuum_message_attachment
Translate-URL: https://translation.odoo-community.org/projects/server-tools-16-0/server-tools-16-0-autovacuum_message_attachment/

Verified

This commit was signed with the committer’s verified signature.

Verified

This commit was signed with the committer’s verified signature.
@GabbasovDinar GabbasovDinar force-pushed the 17.0-t4316-autovacuum_message_attachment-mig branch from 1f5f226 to 5c03d4e Compare February 11, 2025 11:27
Copy link

@mike-cetmix mike-cetmix left a comment

Choose a reason for hiding this comment

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

Functionlity LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet