Skip to content

[16.0][MIG] storage_backend_s3: Migration to 16.0#550

Closed
IJOL wants to merge 50 commits intoOCA:16.0from
IJOL:16.0-mig-storage_backend_s3
Closed

[16.0][MIG] storage_backend_s3: Migration to 16.0#550
IJOL wants to merge 50 commits intoOCA:16.0from
IJOL:16.0-mig-storage_backend_s3

Conversation

@IJOL
Copy link

@IJOL IJOL commented Feb 3, 2026

Summary

Migration of storage_backend_s3 from 15.0 to 16.0

Changes

  • Updated version to 16.0.1.0.0
  • Removed boto3 version constraint (was <=1.15.18)
  • Keep attrs syntax (Odoo 16 still requires it, same as storage_backend_sftp)
  • Tests use VCR cassettes for S3 API mocking

Migration Notes

This module provides Amazon S3 storage backend support.
The attrs syntax is kept as it's still required in Odoo 16.0 - the direct attribute syntax (invisible="expr") causes validation errors.

Testing

  • Pre-commit hooks pass
  • CI should pass with existing VCR cassettes

@IJOL IJOL force-pushed the 16.0-mig-storage_backend_s3 branch 5 times, most recently from 905862a to 2250c12 Compare February 3, 2026 18:36
sebastienbeau and others added 25 commits February 3, 2026 19:37
- refactor the way to build the url (use a generic base_url).
- make more generic the storage backend by moving specific feature in
  storage file
- better name for variable "name" in store and retrieve method use
  "relative_path" instead
- extra amazon S3 storage component in a separated module with test
  using vcrpy
…t with the specifiation of the type of file binary or base64
* support custom endpoint
* refactor bucket handling
* re-register vcrpy tests
@IJOL IJOL force-pushed the 16.0-mig-storage_backend_s3 branch from 2250c12 to feff193 Compare February 3, 2026 18:40
@IJOL IJOL force-pushed the 16.0-mig-storage_backend_s3 branch from 39ef25a to 9bae181 Compare February 3, 2026 18:53
@IJOL
Copy link
Author

IJOL commented Feb 3, 2026

Closing to recreate from BITVAX/storage organization fork. Same changes will be submitted in a new PR.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.