-
-
Notifications
You must be signed in to change notification settings - Fork 163
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
[ADD] attachment_db_by_checksum #239
[ADD] attachment_db_by_checksum #239
Conversation
Is any of the previous reviewers still interested? @gurneyalex @simahawk |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks
@OCA/storage-maintainers maybe someone is interested in this module? |
- while deleting attachment content, search all the attachments, otherwise we delete content still used - checksum is longer than 40; update its content
fd7203c
to
5e02901
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I guess this is used in prod since a while now, right? :)
This PR has the |
/ocabot merge nobump |
Hey, thanks for contributing! Proceeding to merge this for you. |
Congratulations, your PR was merged at 22f11fc. Thanks a lot for contributing to OCA. ❤️ |
Third attempt to share this module with the OCA ecosystem, after the previous attempts have gone stale:
10.0
: [10.0] attachment_db_by_checksum #10912.0
: [12.0] ADD attachment_db_by_checksum #107Here I am preserving the history of #107 (following https://github.com/OCA/maintainer-tools/wiki/Migration-to-version-14.0), and have added a few goodies:
Hoping the third time's a charm 🍀