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] project_template: Migration to 13.0 #684

Merged
merged 22 commits into from Jul 9, 2020
Merged

[MIG] project_template: Migration to 13.0 #684

merged 22 commits into from Jul 9, 2020

Conversation

ghost
Copy link

@ghost ghost commented Jun 18, 2020

No description provided.

patrickrwilson and others added 21 commits June 18, 2020 12:32
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/
@OCA-git-bot
Copy link
Contributor

Hi @patrickrwilson,
some modules you are maintaining are being modified, check this out!

@ghost ghost mentioned this pull request Jun 18, 2020
35 tasks
Copy link

@HaraldPanten HaraldPanten left a comment

Choose a reason for hiding this comment

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

Hi @Alfadil

It seems that it exists an incompatibility with project_key module. I describe the behaviour:

1- Open the options of a project and set "is template" checkbox to True.
2- Once the project has been defined as a template, click to "Create project from template". It doesn't matter if you do that by the form view or the Kanban view. It will work properly.
3- Try to create a second project using the same template as before: An error occurs as a restriction popup will tell the user that "Project key must be unique".

If you create one project from a template it is working fine. If you try to create a second project using the same template, user is going to be restricted by the unique key.

Could you check?

Thanks!

@ghost
Copy link
Author

ghost commented Jun 21, 2020

hi @HaraldPanten
the incompatibility with project_key module is inherited problem from previous versions, And frankly I think the user should change the name of the project generated from project template rather than trying to change the behavior of one of the modules
Thanks for your comment

@ghost ghost requested a review from HaraldPanten June 21, 2020 12:27
Copy link

@HaraldPanten HaraldPanten left a comment

Choose a reason for hiding this comment

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

It's OK then.

Functional review 👍

Copy link
Contributor

@patrickrwilson patrickrwilson left a comment

Choose a reason for hiding this comment

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

LGTM

@OCA-git-bot
Copy link
Contributor

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

@AaronHForgeFlow
Copy link
Contributor

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 13.0-ocabot-merge-pr-684-by-AaronHForgeFlow-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 0d72cc1 into OCA:13.0 Jul 9, 2020
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at a03fe15. 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