psa-systems/mokosh-apps
Watch
9
0
Fork
You've already forked mokosh-apps
 0
Code Issues Pull requests 1 Projects Releases 14 Packages Wiki Activity Actions

fix(time): bound Hours input to positive decimals up to 24h #137

Merged
David merged 2 commits from fix/mapps-150-hours-decimal-bounds into main 2026-06-14 15:57:33 +02:00
Conversation 0 Commits 2 Files changed 2 +23 -2
David commented 2026-06-14 15:42:13 +02:00
Owner
Copy link

The Log Time Hours input carried step="0.25" but no min/max, so the
browser accepted negative and unbounded values (e.g. -1, 1e3). Add min
and max props to the shared Input component and apply min="0.25"/max="24"
to the Hours field, mirroring the bound in the submit-time validation so
non-browser submits are rejected too. HH:MM input is not supported;
decimal hours only.

#MAPPS-150

The Log Time Hours input carried step="0.25" but no min/max, so the browser accepted negative and unbounded values (e.g. -1, 1e3). Add min and max props to the shared Input component and apply min="0.25"/max="24" to the Hours field, mirroring the bound in the submit-time validation so non-browser submits are rejected too. HH:MM input is not supported; decimal hours only. #MAPPS-150
fix(time): bound Hours input to positive decimals up to 24h
All checks were successful
Check / clippy + fmt + tests (pull_request) Successful in 2m4s
Details
0f7f67b831 
The Log Time Hours input carried step="0.25" but no min/max, so the
browser accepted negative and unbounded values (e.g. -1, 1e3). Add min
and max props to the shared Input component and apply min="0.25"/max="24"
to the Hours field, mirroring the bound in the submit-time validation so
non-browser submits are rejected too. HH:MM input is not supported;
decimal hours only.

#MAPPS-150
Merge branch 'main' into fix/mapps-150-hours-decimal-bounds
All checks were successful
Create release / Create release from merged PR (pull_request) Has been skipped
Details
Check / clippy + fmt + tests (pull_request) Successful in 3m28s
Details
74ad08ee82
David merged commit a8e9759fec into main 2026-06-14 15:57:33 +02:00
David deleted branch fix/mapps-150-hours-decimal-bounds 2026-06-14 15:57:33 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
psa-systems/mokosh-apps!137
No description provided.