Skip to content
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

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

Merged
merged 7 commits into from
Sep 22, 2023

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
6 checks passed
@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