[18.0][IMP] storage_backend: add storage category + limit swap by category#633
Conversation
OCA-git-bot
added
series:18.0
mod:storage_file
simahawk
changed the title
simahawk
force-pushed
the
18-backend-category
branch
3 times, most recently
from
0a2e3f7 to
0cb1fd2
Compare
OCA-git-bot
added
mod:storage_media
|
Is this work ready for review? I would suggest renaming the PR to indicate that the changes affect several addons. Also, the commit history could benefit from some squashing. |
rrebollo
left a comment
rrebollo
left a comment
There was a problem hiding this comment.
Pretty solid development. I would suggest mentioning the feature in some of the addon's documentation. Also, please address the commit organization issues. Once that's done, feel free to ping me and I'll approve.
Also, please consider reviewing my contribution (#634) in return.
There's no need to mention all the addons:
Squashing will be done at the end. |
A way to classify storages in a functional way. For instance: you might have N storage backends for the shop, N storage backends for data import, N storage backends for EDI purposes. With this change you can finally organize them and let your customers understand what such storages do.
Is not allowed to swap files to a backend of a different category. A technical door is left opened in case for specific reasons you want force file swap by using the ctx key swap_backend_bypass_category_check.
simahawk
force-pushed
the
18-backend-category
branch
from
0cb1fd2 to
7e3ca26
Compare
|
/ocabot merge minor |
|
Hey, thanks for contributing! Proceeding to merge this for you. |
|
Congratulations, your PR was merged at 7c21840. Thanks a lot for contributing to OCA. ❤️ |
4 participants
A way to classify storages in a functional way.
For instance: you might have N storage backends for the shop,
N storage backends for data import, N storage backends for EDI purposes.
With this change you can finally organize them
and let your customers understand what such storages do.
On top of that, swapping files is now limited by category.