- Sponsor
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
base: 17.0
Are you sure you want to change the base?
[MIG] autovacuum_message_attachment: Migration to 17.0 #3183
Conversation
… it is already present in other models
There was a problem hiding this 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:
The changes so fare LGTM.
@CRogos Updated the readme to make the module behaviour more clear |
@Bearnard21 I apricate your work. Could you also do the squashing? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code LGTM!
This PR has the |
05a85c5
to
1f5f226
Compare
@CRogos Hi, I squashed some commits |
There are still weblate and oca-ci commits which can be squashed. |
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/
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.
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/
But allow it only if a file pattern is set, to avoid deleting all orphan attachments are some are auto-generated by Odoo
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/
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/
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/
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/
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/
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/
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/
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/
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/
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/
1f5f226
to
5c03d4e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Functionlity LGTM
No description provided.