Skip to content
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

[17.0][MIG] data_encryption: Migration to 17.0 #198

Open
wants to merge 31 commits into
base: 17.0
Choose a base branch
from

Conversation

map-240221
Copy link

No description provided.

florian-dacosta and others added 30 commits August 19, 2024 09:36
This module come from the previous module keychain. It has been heavely refactored
to be compatible with server_environment module. For this it has been split into
2 modules data_encryption and server_environment_data_encryption
…to 14.0

note: exclude tests/fixtures from pre-commit check
Currently translated at 45.0% (9 of 20 strings)

Translation: server-env-16.0/server-env-16.0-data_encryption
Translate-URL: https://translation.odoo-community.org/projects/server-env-16-0/server-env-16-0-data_encryption/de/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: server-env-16.0/server-env-16.0-data_encryption
Translate-URL: https://translation.odoo-community.org/projects/server-env-16-0/server-env-16-0-data_encryption/
Currently translated at 100.0% (20 of 20 strings)

Translation: server-env-16.0/server-env-16.0-data_encryption
Translate-URL: https://translation.odoo-community.org/projects/server-env-16-0/server-env-16-0-data_encryption/es/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: server-env-16.0/server-env-16.0-data_encryption
Translate-URL: https://translation.odoo-community.org/projects/server-env-16-0/server-env-16-0-data_encryption/
Currently translated at 100.0% (20 of 20 strings)

Translation: server-env-16.0/server-env-16.0-data_encryption
Translate-URL: https://translation.odoo-community.org/projects/server-env-16-0/server-env-16-0-data_encryption/it/
@map-240221 map-240221 mentioned this pull request Aug 19, 2024
6 tasks
@mkrnac
Copy link

mkrnac commented Sep 25, 2024

when this PR can be merged?

@pedrobaeza
Copy link
Member

It requires reviews: https://github.com/OCA/odoo-community.org/blob/master/website/Contribution/CONTRIBUTING.rst#review

Copy link

@florian-dacosta florian-dacosta left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM code review.
I think you could squash the 2 last commits together though

@map-240221
Copy link
Author

Squashed the commits into one

@florian-dacosta
Copy link

Thanks

@moitabenfdz @bguillot Could you review please?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.