[17.0] [IMP] storage: Remove dependency on server_environment - #654
[17.0] [IMP] storage: Remove dependency on server_environment#654yankinmax wants to merge 19 commits into
Conversation
5077001 to
959ad04
Compare
|
Hello @etobella @lmignon @ivantodorovich |
…lumns on uninstall
… columns on uninstall
…ore columns on uninstall
…re columns on uninstall
…restore columns on uninstall
959ad04 to
4a5bf2a
Compare
abb1082 to
04b1706
Compare
| "depends": ["fs_storage", "server_environment"], | ||
| "post_init_hook": "post_init_hook", | ||
| "uninstall_hook": "uninstall_hook", | ||
| "auto_install": True, |
There was a problem hiding this comment.
Hmm are we sure we want to make these modules auto-install?
That means that if I still have server env installed for a different app (eg: mail) this module will be installed as well w/o any reason.
Since the migration is taking care of installing the modules I would say this has to change.
Am I missing any particular reason to keep auto-install?
This is a forward-port of:
With the applied fixes introduced in: