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

[MIG] base_tier_validation_formula: Migrate to version 17.0 #799

Merged
merged 39 commits into from
Jan 25, 2024

Conversation

sonhd91
Copy link

@sonhd91 sonhd91 commented Jan 10, 2024

@sonhd91 sonhd91 force-pushed the 17.0-mig-base_tier_validation_formula branch from 0539c98 to 39c4544 Compare January 10, 2024 05:43
Copy link
Contributor

@bosd bosd left a comment

Choose a reason for hiding this comment

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

Thanks for this pr! Looks good, Some non-blocking comments.

"name": "Base Tier Validation Formula",
"summary": """
Formulas for Base tier validation""",
"version": "17.0.1.0.0",
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
"version": "17.0.1.0.0",
"version": "17.0.1.0.0",
"category": "Tools",

"summary": """
Formulas for Base tier validation""",
"version": "17.0.1.0.0",
"license": "AGPL-3",
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
"license": "AGPL-3",
"license": "AGPL-3",
"development_status": "Mature",

Copy link
Contributor

Choose a reason for hiding this comment

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

Promotion should'nt have to be in migration commit. But I noticed the missing tag, and it likely satisfies mature status.

}
)

# Create users:
Copy link
Contributor

Choose a reason for hiding this comment

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

Is there a reason new users are created here?
(It comes at a cost, can the users from the base module be re-used?)

Copy link
Author

@sonhd91 sonhd91 Jan 10, 2024

Choose a reason for hiding this comment

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

Yeah, that will do, but we still need create one more user for tests case

AdriaGForgeFlow and others added 28 commits January 10, 2024 14:39
fixup and extend tests

[ADD] systray icon for pending reviews

[FIX] Remove python safe_eval

[ADD] base_tier_validation_formula and migration scripts

[ADD] widget domain and python expression to define reviewer in tier definition

[ADD] auto updating of systray icon counter

[ADD] validation date field

[ADD] review widget dropdown menu
Currently translated at 100.0% (17 of 17 strings)

Translation: server-ux-12.0/server-ux-12.0-base_tier_validation_formula
Translate-URL: https://translation.odoo-community.org/projects/server-ux-12-0/server-ux-12-0-base_tier_validation_formula/es/
Currently translated at 100.0% (17 of 17 strings)

Translation: server-ux-12.0/server-ux-12.0-base_tier_validation_formula
Translate-URL: https://translation.odoo-community.org/projects/server-ux-12-0/server-ux-12-0-base_tier_validation_formula/zh_CN/
[UPD] Update base_tier_validation_formula.pot
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: server-ux-13.0/server-ux-13.0-base_tier_validation_formula
Translate-URL: https://translation.odoo-community.org/projects/server-ux-13-0/server-ux-13-0-base_tier_validation_formula/
… in tier definitions

[UPD] Update base_tier_validation_formula.pot
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: server-ux-13.0/server-ux-13.0-base_tier_validation_formula
Translate-URL: https://translation.odoo-community.org/projects/server-ux-13-0/server-ux-13-0-base_tier_validation_formula/
Currently translated at 100.0% (15 of 15 strings)

Translation: server-ux-13.0/server-ux-13.0-base_tier_validation_formula
Translate-URL: https://translation.odoo-community.org/projects/server-ux-13-0/server-ux-13-0-base_tier_validation_formula/zh_CN/

Translated using Weblate (Chinese (Simplified))

Currently translated at 100.0% (15 of 15 strings)

Translation: server-ux-13.0/server-ux-13.0-base_tier_validation_formula
Translate-URL: https://translation.odoo-community.org/projects/server-ux-13-0/server-ux-13-0-base_tier_validation_formula/zh_CN/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: server-ux-13.0/server-ux-13.0-base_tier_validation_formula
Translate-URL: https://translation.odoo-community.org/projects/server-ux-13-0/server-ux-13-0-base_tier_validation_formula/
Currently translated at 80.0% (12 of 15 strings)

Translation: server-ux-13.0/server-ux-13.0-base_tier_validation_formula
Translate-URL: https://translation.odoo-community.org/projects/server-ux-13-0/server-ux-13-0-base_tier_validation_formula/it/
[UPD] Update base_tier_validation_formula.pot
Currently translated at 100.0% (15 of 15 strings)

Translation: server-ux-14.0/server-ux-14.0-base_tier_validation_formula
Translate-URL: https://translation.odoo-community.org/projects/server-ux-14-0/server-ux-14-0-base_tier_validation_formula/es/
JasminSForgeFlow and others added 10 commits January 10, 2024 14:39
[UPD] Update base_tier_validation_formula.pot
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

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

Translation: server-ux-16.0/server-ux-16.0-base_tier_validation_formula
Translate-URL: https://translation.odoo-community.org/projects/server-ux-16-0/server-ux-16-0-base_tier_validation_formula/
Currently translated at 80.0% (12 of 15 strings)

Translation: server-ux-16.0/server-ux-16.0-base_tier_validation_formula
Translate-URL: https://translation.odoo-community.org/projects/server-ux-16-0/server-ux-16-0-base_tier_validation_formula/it/
- Include context keys for avoiding mail operations overhead.
@sonhd91 sonhd91 force-pushed the 17.0-mig-base_tier_validation_formula branch from 39c4544 to 2c86da9 Compare January 10, 2024 08:44
@sonhd91
Copy link
Author

sonhd91 commented Jan 10, 2024

Branch updated, please check again

Since base_tier_validation was merged, I uncomment some related code for this module

@sonhd91 sonhd91 force-pushed the 17.0-mig-base_tier_validation_formula branch from 2c86da9 to c2499af Compare January 10, 2024 08:47
@sonhd91 sonhd91 mentioned this pull request Jan 10, 2024
22 tasks
@gurneyalex gurneyalex added this to the 17.0 milestone Jan 12, 2024
@gurneyalex
Copy link
Member

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 17.0-ocabot-merge-pr-799-by-gurneyalex-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 4947f97 into OCA:17.0 Jan 25, 2024
7 checks passed
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at 782d799. Thanks a lot for contributing to OCA. ❤️

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

Successfully merging this pull request may close these issues.

None yet