Skip to content

[Enhancement] Sum plan hours before rounding to avoid cumulative drift #187

@EdiWeeks

Description

@EdiWeeks

Description

In PlanEntryModal fetchExistingLines, rounding happens after each line is added (existingVal + hoursValue). If there are multiple lines for the same day and their converted values aren't exact 0.25h multiples, repeated rounding can skew the final sum.

Expected Behavior

Sum all raw hours for a date first, then round once at the end for display.

Context

Raised from Copilot review on PR #184.

Metadata

Metadata

Assignees

Labels

No labels
No labels

Type

No type
No fields configured for issues without a type.

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions