Skip to content

[16.0][ADD] fs_image: New field to store your images into external filesystems#274

Merged
OCA-git-bot merged 7 commits intoOCA:16.0from
acsone:16.0-add-fs-image
Sep 22, 2023
Merged

[16.0][ADD] fs_image: New field to store your images into external filesystems#274
OCA-git-bot merged 7 commits intoOCA:16.0from
acsone:16.0-add-fs-image

Conversation

@lmignon
Copy link
Contributor

@lmignon lmignon commented Aug 25, 2023

  • Tests related and resize
  • Add widget
  • Allows to update alt_name only with dict

Fix image streaming
Fix image streaming with resize
Fix content size whithout x-send-file
Uses the FilUploader component to manage the file selection and save the file content at same time as the model. Prior to this change, the content was saved as soon as it was selected which is not possible on new records
* Defines extension hooks to use to specify the object type to use as field value and the value to pass when updating the attachment.
* Fixes onchange
@lmignon lmignon marked this pull request as ready for review September 3, 2023 13:45
@lmignon lmignon changed the title [16.0][ADD] fs_image: wip.... [16.0][ADD] fs_image: New field to store image into external filesystems Sep 3, 2023
@lmignon lmignon changed the title [16.0][ADD] fs_image: New field to store image into external filesystems [16.0][ADD] fs_image: New field to store your images into external filesystems Sep 3, 2023
Store your image into an external filesystem
This code was initially used to magically know if we need to obfusc or not the filename into the database. Since this option is now explicit, we can safely remove-it. Moreover, without this code it's now possible to migrate old storage_file contents to the new fs_* addons
@lmignon
Copy link
Contributor Author

lmignon commented Sep 22, 2023

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 16.0-ocabot-merge-pr-274-by-lmignon-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 97627c2 into OCA:16.0 Sep 22, 2023
@OCA-git-bot
Copy link
Contributor

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

@lmignon lmignon deleted the 16.0-add-fs-image branch September 29, 2023 12:15
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.

2 participants

Comments