-
-
Notifications
You must be signed in to change notification settings - Fork 872
[19.0][MIG] project_template: Migration to 19.0 #1617
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
base: 19.0
Are you sure you want to change the base?
[19.0][MIG] project_template: Migration to 19.0 #1617
Conversation
Adding new Project Template Module [FIX] PEP8 Errors Revert "[FIX] PEP8 Errors" This reverts commit 90f9cd9. [FIX] Fixed PEP8 Formatting First upload tests showed some lint issues which this commit has the fixes for. [FIX] Fixed LINT Issue Removed unreachable code that was not needed. [FIX] RunBot Warning Fixed runbot warning about @Class to hasclass [UPD] Suggested Changes Updated based on suggested changes on other submitted modules. [UPD] Manifest and Readme Updated Manifest and Readme based on comments on other module pr's. [UPD] Suggested Changes Updated code based on suggested code changes from reviewer. [FIX] Creation Error Fixed create project error when creating project via configuration/projects menu [UPD] Menu and Bool Change Removed Template Menu (Filter option still there) and moved Bool up below name. (From reviewer suggestions) [UPD] Kanban Menu Removed button, added link to kanban menu. Start of tests [UPD] Remove project_milestone code Removed the project_milestone code and will create a separate module for that purpose.
Currently translated at 100.0% (7 of 7 strings) Translation: project-12.0/project-12.0-project_template Translate-URL: https://translation.odoo-community.org/projects/project-12-0/project-12-0-project_template/es/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: project-12.0/project-12.0-project_template Translate-URL: https://translation.odoo-community.org/projects/project-12-0/project-12-0-project_template/
There is an error if the template name gets modified so needed to add a new else statement for the new_name.
Currently translated at 100.0% (7 of 7 strings) Translation: project-12.0/project-12.0-project_template Translate-URL: https://translation.odoo-community.org/projects/project-12-0/project-12-0-project_template/pt/
Currently translated at 85.7% (6 of 7 strings) Translation: project-12.0/project-12.0-project_template Translate-URL: https://translation.odoo-community.org/projects/project-12-0/project-12-0-project_template/fr/
Currently translated at 100.0% (7 of 7 strings) Translation: project-12.0/project-12.0-project_template Translate-URL: https://translation.odoo-community.org/projects/project-12-0/project-12-0-project_template/sl/
Currently translated at 14.3% (1 of 7 strings) Translation: project-12.0/project-12.0-project_template Translate-URL: https://translation.odoo-community.org/projects/project-12-0/project-12-0-project_template/it/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: project-13.0/project-13.0-project_template Translate-URL: https://translation.odoo-community.org/projects/project-13-0/project-13-0-project_template/
Currently translated at 100.0% (7 of 7 strings) Translation: project-13.0/project-13.0-project_template Translate-URL: https://translation.odoo-community.org/projects/project-13-0/project-13-0-project_template/he/
Currently translated at 100.0% (10 of 10 strings) Translation: project-16.0/project-16.0-project_template Translate-URL: https://translation.odoo-community.org/projects/project-16-0/project-16-0-project_template/de/
Currently translated at 100.0% (7 of 7 strings) Translation: project-16.0/project-16.0-project_template Translate-URL: https://translation.odoo-community.org/projects/project-16-0/project-16-0-project_template/hr/
Currently translated at 100.0% (7 of 7 strings) Translation: project-16.0/project-16.0-project_template Translate-URL: https://translation.odoo-community.org/projects/project-16-0/project-16-0-project_template/it/
Currently translated at 100.0% (7 of 7 strings) Translation: project-16.0/project-16.0-project_template Translate-URL: https://translation.odoo-community.org/projects/project-16-0/project-16-0-project_template/pt_BR/
Currently translated at 85.7% (6 of 7 strings) Translation: project-16.0/project-16.0-project_template Translate-URL: https://translation.odoo-community.org/projects/project-16-0/project-16-0-project_template/nl/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: project-16.0/project-16.0-project_template Translate-URL: https://translation.odoo-community.org/projects/project-16-0/project-16-0-project_template/
Previously, if multiple tasks in a template project had the same name, their end_date-values were not copied properly. Co-authored-by: Stefan Rijnhart <stefan@opener.amsterdam>
* Change the way to copy end dates to tasks due to api change to _map_tasks_default_values which now longer gets passed the task * Refactor tests to use setUpClass (finally) * Update overwritten action context with upstream changes * Remove originally unmodified (but now outdated) fields from overwritten action * Update kanban element selector
|
/ocabot migration project_template |
hieulucky111
left a comment
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.
LGTM!
huan-tq
left a comment
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.
LGTM
Quan-nhm
left a comment
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.
Functional Test and Code review LGTM!
|
This PR has the |
|
Hello @Vang-NguyenPhu could you quickly take the time to reduce a bit the administrative commits? https://github.com/OCA/maintainer-tools/wiki/Merge-commits-in-pull-requests#mergesquash-the-commits-generated-by-bots-or-weblate |
No description provided.