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

[15.0][ADD] project_task_required_project: project task required project. #1417

Merged

Conversation

angelgarciadelachica
Copy link

This module obliges you to select a project when creating or editing a project task.

[T-7322]

@angelgarciadelachica angelgarciadelachica changed the title [ADD] project_task_required_project: project task required project. [15.0][ADD] project_task_required_project: project task required project. Jan 20, 2025
@angelgarciadelachica angelgarciadelachica force-pushed the 15.0-add-project_task_required_project branch 3 times, most recently from 4a12f9b to 450b93f Compare January 20, 2025 14:56
@angelgarciadelachica
Copy link
Author

@HaraldPanten @luis-ron Can you review this PR?
THX!!!

@HaraldPanten
Copy link

@ValentinVinagre

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.

Some comments.

project_task_required_project/models/project_task.py Outdated Show resolved Hide resolved
Copy link

@ValentinVinagre ValentinVinagre left a comment

Choose a reason for hiding this comment

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

some comments


@api.constrains("project_id")
def _check_project_id_not_null(self):
if self.env.company.is_project_required and any(

This comment was marked as resolved.

@ValentinVinagre
Copy link

@manuelregidor can you review it?

@angelgarciadelachica angelgarciadelachica force-pushed the 15.0-add-project_task_required_project branch 2 times, most recently from b34d31d to c7c1e4c Compare January 21, 2025 10:56
@angelgarciadelachica angelgarciadelachica force-pushed the 15.0-add-project_task_required_project branch from c7c1e4c to 182c52f Compare January 21, 2025 12:45
Copy link
Contributor

@manuelregidor manuelregidor left a comment

Choose a reason for hiding this comment

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

Technical review. LGTM

Copy link

@ValentinVinagre ValentinVinagre left a comment

Choose a reason for hiding this comment

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

LGTM 👍🏻

@angelgarciadelachica angelgarciadelachica force-pushed the 15.0-add-project_task_required_project branch from 182c52f to e439ee1 Compare January 21, 2025 15:13
@HaraldPanten
Copy link

@leemannd Is this PR ready now? THX for reviewing!

@leemannd
Copy link

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 15.0-ocabot-merge-pr-1417-by-leemannd-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 1b72d21 into OCA:15.0 Jan 23, 2025
7 checks passed
@OCA-git-bot
Copy link
Contributor

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

@HaraldPanten HaraldPanten deleted the 15.0-add-project_task_required_project branch January 23, 2025 15:13
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.

6 participants