Skip to content

[15.0][MIG] attachment_zipped_download: Migration to 15.0 - #473

Merged
OCA-git-bot merged 8 commits into
OCA:15.0from
Tecnativa:15.0-mig-attachment_zipped_download
Apr 2, 2024
Merged

[15.0][MIG] attachment_zipped_download: Migration to 15.0#473
OCA-git-bot merged 8 commits into
OCA:15.0from
Tecnativa:15.0-mig-attachment_zipped_download

Conversation

@victoralmau

Copy link
Copy Markdown
Member

Migration to 15.0

Please @pedrobaeza and @chienandalu can you review it?

@Tecnativa TT48578

victoralmau and others added 8 commits April 2, 2024 13:55
TT33403

[UPD] Update attachment_zipped_download.pot

[UPD] README.rst
Translated using Weblate (Italian)

Currently translated at 100.0% (7 of 7 strings)

Translation: knowledge-14.0/knowledge-14.0-attachment_zipped_download
Translate-URL: https://translation.odoo-community.org/projects/knowledge-14-0/knowledge-14-0-attachment_zipped_download/it/
attachment_zipped_download 14.0.1.0.1
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 14.0.1.0.2
…ad mixin

This helps to download multiple attachments from any models.

[UPD] Update attachment_zipped_download.pot

[UPD] README.rst

attachment_zipped_download 14.0.2.0.0

[UPD] README.rst

Update translation files

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

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

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

Update translation files

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

Translation: knowledge-14.0/knowledge-14.0-attachment_zipped_download
Translate-URL: https://translation.odoo-community.org/projects/knowledge-14-0/knowledge-14-0-attachment_zipped_download/

Translated using Weblate (Spanish)

Currently translated at 100.0% (10 of 10 strings)

Translation: knowledge-14.0/knowledge-14.0-attachment_zipped_download
Translate-URL: https://translation.odoo-community.org/projects/knowledge-14-0/knowledge-14-0-attachment_zipped_download/es/

@chienandalu chienandalu left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Test and code review 👍

@HaraldPanten

Copy link
Copy Markdown
Contributor

/ocabot migration attachment_zipped_download

@OCA-git-bot OCA-git-bot added this to the 15.0 milestone Apr 2, 2024
@OCA-git-bot OCA-git-bot mentioned this pull request Apr 2, 2024
11 tasks

@pedrobaeza pedrobaeza left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

/ocabot merge nobump

@OCA-git-bot

Copy link
Copy Markdown
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 15.0-ocabot-merge-pr-473-by-pedrobaeza-bump-nobump, awaiting test results.

@OCA-git-bot
OCA-git-bot merged commit efc99fe into OCA:15.0 Apr 2, 2024
@OCA-git-bot

Copy link
Copy Markdown
Contributor

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

@pedrobaeza
pedrobaeza deleted the 15.0-mig-attachment_zipped_download branch April 2, 2024 18:10
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.

7 participants