Repository navigation
Aditya Dhayapulay - Split timer time across assigned tasks - #5784
Open
adit24dhaya wants to merge 2 commits into
Open
adit24dhaya wants to merge 2 commits into
adit24dhaya wants to merge 2 commits into
Conversation
Implement the Phase I Split One Timer Session Across Multiple Assigned Tasks request. Let users allocate the session total with separate task descriptions, preserve uncertain saves for safe retry and confirm timer completion without reposting entries. Task: HGN Phase I Bugs and Needed Functionalities, Split One Timer Session Across Multiple Assigned Tasks Validation: 46 frontend tests, 13 browser cases, production build and changed-file JS/CSS lint.
✅ Deploy Preview for highestgoodnetwork-dev ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Use native status outputs, clarify input label spacing, simplify submit labels and separate form presentation helpers to reduce cognitive complexity. Preserve the existing single-task behavior and split recovery flow. Validation: 46 passing frontend tests, 13 browser cases, and JS/CSS lint.
adit24dhaya
marked this pull request as ready for review
November 5, 2026 22:31
|
one-community
added
the
High Priority - Please Review First
This branch has not been deployed
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Description
Add an optional Worked on multiple tasks? flow to Add Tangible Time Entry. Users can divide the timer total between assigned tasks, with separate descriptions and reference links. The unchecked default keeps the existing single-entry form.
Original task request
HGN Phase I Bugs and Needed Functionalities — Split One Timer Session Across Multiple Assigned Tasks
Original assignment: (PRIORITY HIGH) (FEATURE REQUEST) Aditya Dhayapulay: Split One Timer Session Across Multiple Assigned Tasks, estimated 15 hours.
Related PRS (if any):
Backend: OneCommunityGlobal/HGNRest#2472. Both task branches are required to test split saves; merge the backend first.
Main changes explained:
How to test:
Validation: 46 frontend tests passed, including existing Timer/TimeEntryForm suites and new recovery/parent-flow checks. 13 browser cases passed across 320/390/768/1280px, both themes, lost responses, stop failure, rejected assignments, receipt recovery and disconnection. Production build, changed-file JS lint and CSS lint passed. Backend validation: 92 passing tests with a disposable local replica set.
Screenshots or videos of changes:
Note:
Splitting is available for the signed-in user's timer entries. Recovery requires browser site storage; the form fails safely before sending if storage is unavailable. Browser tests use real form components with synthetic tasks and mocked API/stop acknowledgements; backend transaction tests use a disposable local MongoDB replica set. No shared data writes, merges or manual deployments were performed.