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

[#58113] 14.6 Documentation: Progress reporting changes https://comm… #16869

Open
wants to merge 10 commits into
base: release/14.6
Choose a base branch
from

Conversation

MayaBerd
Copy link
Contributor

@MayaBerd MayaBerd commented Oct 1, 2024

…unity.openproject.org/work_packages/58113

[#58113] 14.6 Documentation: Progress reporting changes

https://community.openproject.org/work_packages/58113

Ticket

What are you trying to accomplish?

Screenshots

What approach did you choose and why?

Merge checklist

  • Added/updated tests
  • Added/updated documentation in Lookbook (patterns, previews, etc)
  • Tested major browsers (Chrome, Firefox, Edge, ...)

@MayaBerd MayaBerd marked this pull request as ready for review October 2, 2024 09:49
Copy link
Member

@cbliard cbliard left a comment

Choose a reason for hiding this comment

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

Nice write up Maya 👏

I fixed some typos here and there and also found 2 places where the text is imprecise or incorrect.

MayaBerd and others added 6 commits October 2, 2024 14:47
…ress-tracking/README.md

Co-authored-by: Christophe Bliard <c.bliard@openproject.com>
…ress-tracking/README.md

Co-authored-by: Christophe Bliard <c.bliard@openproject.com>
Co-authored-by: Christophe Bliard <c.bliard@openproject.com>
Co-authored-by: Christophe Bliard <c.bliard@openproject.com>
Co-authored-by: Christophe Bliard <c.bliard@openproject.com>
Co-authored-by: Christophe Bliard <c.bliard@openproject.com>
@@ -162,7 +162,7 @@ OpenProject offers two modes for calculating *% Complete* in hierarchy totals:
>
> Work packages without *Work* will be ignored.

- **Simple average**: *Work* is ignored and the *total % Complete* is calculated as a simple average of the *% Complete* values from the work packages in the hierarchy.
- **Simple average**: *Work* is ignored and the *total % Complete* is calculated as a simple average of the *% Complete* values from the direct work packages children in the hierarchy. The value used on each direct child for computing the average is its *total % Complete* value, or its *% Complete* value if it has no children, or 0% if its *% Complete* value is not set.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
- **Simple average**: *Work* is ignored and the *total % Complete* is calculated as a simple average of the *% Complete* values from the direct work packages children in the hierarchy. The value used on each direct child for computing the average is its *total % Complete* value, or its *% Complete* value if it has no children, or 0% if its *% Complete* value is not set.
- **Simple average**: *Work* is ignored and the *total % Complete* is calculated as a simple average of the *% Complete* values from the direct work packages children in the hierarchy. The value used on each direct child for computing the average is its *total % Complete* value, or its *% Complete* value if it has no children, or 0% if its *% Complete* value is empty.

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

Successfully merging this pull request may close these issues.

2 participants