-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
[#57529] Add options for sum totals calculation mode in hierarchies #16649
Merged
cbliard
merged 23 commits into
dev
from
implementation/57529-options-for-total-calculation-sum-mode
Sep 26, 2024
Merged
Changes from 1 commit
Commits
Show all changes
23 commits
Select commit
Hold shift + click to select a range
291b87b
Create setting for Calculation of % Complete hierarchy totals
aaron-contreras b92e9d1
Incorporate simple average mode into total % complete calculations
aaron-contreras 53a3a72
Adjust selection of done ratio based on new rules for children
aaron-contreras 793d758
Add radio group to progress tracking form
aaron-contreras 06bd83f
Add feature spec for setting toggling
aaron-contreras 503ad15
Add recalculation in job when switching to work weighted mode
aaron-contreras 5e36ce2
Re-order radio group options according to design document
aaron-contreras 6cbfdc8
Trigger the recalculation job when changing to work weighted mode
aaron-contreras 2f2561c
Add in simple average calculation mode job
aaron-contreras 1f12393
Account for status exclusion from total calculations in simple averag…
aaron-contreras 1874fb3
Fix `status_excluded_from_totals` not existing for `WorkPackage`
cbliard f040848
Fix when all work packages of a hierarchy are excluded from totals
cbliard 6d9a9c8
Removed flaky spec
cbliard 447a6de
Fix rubocop warnings Metrics/AbcSize and Metrics/PerceivedComplexity
cbliard 0f9be1b
Add a test when all hierarchy is excluded from totals
cbliard a574b2f
Add failing spec about handling simple average mode in a job
cbliard 3f97171
Use work_package_hierarchies to create depth table
cbliard 3250f51
Compute total % complete according to mode in update status job
cbliard 946b57d
Merge branch 'dev' into implementation/57529-options-for-total-calcul…
aaron-contreras 0edb100
Add extra spec specifically accounting for excluded statuses
aaron-contreras 5b1dbb4
Test for unset total values
aaron-contreras 95cda6d
Remove unnecessary update
aaron-contreras de86221
Extract common operations to sql_commands
aaron-contreras File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
Nice!