[ADD] dms_storage #182
[ADD] dms_storage #182
Conversation
|
ping @augustodinizl |
| _inherit = "dms.storage" | ||
|
|
||
| save_type = fields.Selection( | ||
| selection_add=[("storage", "Storage")], ondelete={"storage": "cascade"} |
There was a problem hiding this comment.
This is really necessary? With this approach, it could not be used as an attachment type storage actually.
There was a problem hiding this comment.
Well, the idea is to store files on a different place. If you use attachment you will store it on you attachment system. With storage you use the functionality of storage in order to store it somewhere different
There was a problem hiding this comment.
You can setup the attachments to be stored on a storage backend, so it doesn't seem to restrict any functionality.
|
@etobella really great PR! |
|
Well, it was something I did on a sprint code. If you are imterested you can finish it 😉 |
|
There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days. |
github-actions
bot
added
the
stale
pedrobaeza
removed
the
stale
|
There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days. |
github-actions
bot
added
the
stale
pedrobaeza
removed
the
stale
|
@etobella can you rebase and fix pre-commit and let's merge this for not losing it, being a new module, it doesn't hurt. |
etobella
force-pushed
the
14.0-add-dms_storage
branch
from
66e848e
to
384c84c
Compare
|
What a great day to merge this nice PR. Let's do it! |
|
Congratulations, your PR was merged at e541909. Thanks a lot for contributing to OCA. ❤️ |
Allows to storage DMS Files on external storages.
TO DO: