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

[16.0][ADD] project_milestone_status: New module project_milestone_st… #1452

Open
wants to merge 1 commit into
base: 16.0
Choose a base branch
from

Conversation

lbarry-apsl
Copy link
Member

This module allows you to have the percentage of the execution of a project.

Execution: It is the percentage between hours of tasks completed and pending tasks
Dedication: It is the percentage between planned hours and effective hours.

These percentages are shown in Milestones and Project Updates of projects.

cc https://helpdesk.apsl.net/ HT01801

@miquelalzanillas @peluko00 @mpascuall @ppyczko @javierobcn @BernatObrador please review

@lbarry-apsl lbarry-apsl force-pushed the 16.0-add-project_milestone_status branch from 6c5f4f1 to 463bcd3 Compare March 20, 2025 16:45
Copy link

@ppyczko ppyczko left a comment

Choose a reason for hiding this comment

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

It would be advisable to add some tests to ensure the module's functionality and stability.

Copy link

@mpascuall mpascuall left a comment

Choose a reason for hiding this comment

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

Functionally working, tested in runboat

@lbarry-apsl lbarry-apsl force-pushed the 16.0-add-project_milestone_status branch from 463bcd3 to 07d25cf Compare March 21, 2025 11:21
Copy link

@peluko00 peluko00 left a comment

Choose a reason for hiding this comment

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

LGTM, tested in runboat and code review

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

Successfully merging this pull request may close these issues.

5 participants