Skip to content

[19.0] [MIG] attachment_zipped_download: Migration to 19.0#579

Merged
OCA-git-bot merged 22 commits intoOCA:19.0from
HeliconiaIO:19.0-mig-attachment_zipped_download
Mar 19, 2026
Merged

[19.0] [MIG] attachment_zipped_download: Migration to 19.0#579
OCA-git-bot merged 22 commits intoOCA:19.0from
HeliconiaIO:19.0-mig-attachment_zipped_download

Conversation

@BhaveshHeliconia
Copy link
Contributor

No description provided.

victoralmau and others added 22 commits December 30, 2025 10:15
TT33403

[UPD] Update attachment_zipped_download.pot

[UPD] README.rst
[UPD] Update attachment_zipped_download.pot

[UPD] README.rst

Update translation files

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

Translation: knowledge-16.0/knowledge-16.0-attachment_zipped_download
Translate-URL: https://translation.odoo-community.org/projects/knowledge-16-0/knowledge-16-0-attachment_zipped_download/
Translated using Weblate (Italian)

Currently translated at 100.0% (4 of 4 strings)

Translation: knowledge-16.0/knowledge-16.0-attachment_zipped_download
Translate-URL: https://translation.odoo-community.org/projects/knowledge-16-0/knowledge-16-0-attachment_zipped_download/it/
attachment_zipped_download 16.0.1.0.1
Currently translated at 100.0% (4 of 4 strings)

Translation: knowledge-16.0/knowledge-16.0-attachment_zipped_download
Translate-URL: https://translation.odoo-community.org/projects/knowledge-16-0/knowledge-16-0-attachment_zipped_download/it/
Translated using Weblate (Croatian)

Currently translated at 100.0% (4 of 4 strings)

Translation: knowledge-16.0/knowledge-16.0-attachment_zipped_download
Translate-URL: https://translation.odoo-community.org/projects/knowledge-16-0/knowledge-16-0-attachment_zipped_download/hr/
The previous code allowed any authenticated to retreive any attachment present
on odoo filesystem. So a WMS user could technically spoken download a zip with
accounting documents.

This implementation is calling attachemnt.check("read") to ensure access
right and use attachemnt.raw attribute to retreive file to archive which
(not test) should works with attachment saved somewhere else than the
local filesystem (s3 storage, pgsql large object storage...).

attachment_zipped_download 16.0.1.0.2
…ad mixin

This helps to download multiple attachments from any models.

[UPD] README.rst

[UPD] Update attachment_zipped_download.pot

[UPD] README.rst

attachment_zipped_download 16.0.2.0.0

[UPD] README.rst

Update translation files

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

Translation: knowledge-16.0/knowledge-16.0-attachment_zipped_download
Translate-URL: https://translation.odoo-community.org/projects/knowledge-16-0/knowledge-16-0-attachment_zipped_download/
Currently translated at 100.0% (7 of 7 strings)

Translation: knowledge-16.0/knowledge-16.0-attachment_zipped_download
Translate-URL: https://translation.odoo-community.org/projects/knowledge-16-0/knowledge-16-0-attachment_zipped_download/es/
Before the fix, 0 bytes would be transferred.
Currently translated at 100.0% (7 of 7 strings)

Translation: knowledge-16.0/knowledge-16.0-attachment_zipped_download
Translate-URL: https://translation.odoo-community.org/projects/knowledge-16-0/knowledge-16-0-attachment_zipped_download/it/

Update translation files

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

Translation: knowledge-16.0/knowledge-16.0-attachment_zipped_download
Translate-URL: https://translation.odoo-community.org/projects/knowledge-16-0/knowledge-16-0-attachment_zipped_download/
[BOT] post-merge updates

[BOT] post-merge updates
Translated using Weblate (Portuguese)

Currently translated at 100.0% (7 of 7 strings)

Translation: knowledge-16.0/knowledge-16.0-attachment_zipped_download
Translate-URL: https://translation.odoo-community.org/projects/knowledge-16-0/knowledge-16-0-attachment_zipped_download/pt/
TT52268

[UPD] Update attachment_zipped_download.pot

[BOT] post-merge updates

Update translation files

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

Translation: knowledge-17.0/knowledge-17.0-attachment_zipped_download
Translate-URL: https://translation.odoo-community.org/projects/knowledge-17-0/knowledge-17-0-attachment_zipped_download/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: knowledge-18.0/knowledge-18.0-attachment_zipped_download
Translate-URL: https://translation.odoo-community.org/projects/knowledge-18-0/knowledge-18-0-attachment_zipped_download/
@BhaveshHeliconia BhaveshHeliconia mentioned this pull request Dec 30, 2025
11 tasks
@pedrobaeza
Copy link
Member

/ocabot migration attachment_zipped_download

@OCA-git-bot OCA-git-bot added this to the 19.0 milestone Dec 30, 2025
Copy link

@gregory-rutecki gregory-rutecki left a comment

Choose a reason for hiding this comment

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

Tested on live Odoo 19.0 instance, LGTM. All CI green.

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

@gregory-rutecki
Copy link

@pedrobaeza @stferraro Hi, gentle ping on this and the remaining knowledge module PRs (#571-#577, #588, #589).

Our clients are waiting to migrate from Odoo v18 to v19, but they need the knowledge module suite to be officially available in the OCA repository before we can proceed. The complete set of modules has been submitted and all pass pre-commit — several already have the "approved" label.

Could we kindly ask for a review and merge when possible? This is blocking the v18 → v19 migration path for us and likely many other Odoo users. Knowledge is a critical module for service companies relying on Odoo for document management.

Thank you for your time and all the work you do maintaining this repo!

Copy link
Member

@pedrobaeza pedrobaeza left a comment

Choose a reason for hiding this comment

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

Nothing prevents you from using the modules in PRs. We do it when we need it.

Anyway, let's merge this one, as already correct and reviewed by me and 2 more.

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 19.0-ocabot-merge-pr-579-by-pedrobaeza-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit d075d1d into OCA:19.0 Mar 19, 2026
7 checks passed
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at 44c09d4. Thanks a lot for contributing to OCA. ❤️

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.