-
-
Notifications
You must be signed in to change notification settings - Fork 167
[18][MIG] server_environment_data_encryption #226
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
[18][MIG] server_environment_data_encryption #226
Conversation
…f the button does not change randomly
…to 14.0 note: exclude tests/fixtures from pre-commit check
Fallback to normal server env behavior and log a warning.
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: server-env-14.0/server-env-14.0-server_environment_data_encryption Translate-URL: https://translation.odoo-community.org/projects/server-env-14-0/server-env-14-0-server_environment_data_encryption/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: server-env-16.0/server-env-16.0-server_environment_data_encryption Translate-URL: https://translation.odoo-community.org/projects/server-env-16-0/server-env-16-0-server_environment_data_encryption/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: server-env-16.0/server-env-16.0-server_environment_data_encryption Translate-URL: https://translation.odoo-community.org/projects/server-env-16-0/server-env-16-0-server_environment_data_encryption/
Currently translated at 100.0% (5 of 5 strings) Translation: server-env-16.0/server-env-16.0-server_environment_data_encryption Translate-URL: https://translation.odoo-community.org/projects/server-env-16-0/server-env-16-0-server_environment_data_encryption/es/
Currently translated at 100.0% (5 of 5 strings) Translation: server-env-16.0/server-env-16.0-server_environment_data_encryption Translate-URL: https://translation.odoo-community.org/projects/server-env-16-0/server-env-16-0-server_environment_data_encryption/it/
…of the running env
5ebef16
to
38b623a
Compare
Code review + test |
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.
Tested with mail_environment, works as expected
/ocabot merge nobump |
What a great day to merge this nice PR. Let's do it! |
Congratulations, your PR was merged at 7d6a476. Thanks a lot for contributing to OCA. ❤️ |
Standard migration
There is some changes in the module because it seems that custom context (
environment
key here) are not passed toget_views
method.Because of this, we can't manage a different key for the ormcache to manage 1 different view per environment.
So the strategy here is to build a single view with all environment buttons/alert with
invisible
attributes depending on the context.