kayg/calternal
Watch
1
0
Fork
You've already forked calternal
 0
Code Issues 191 Pull requests 61 Projects Releases Packages Wiki Activity Actions 20

Log times: forgive typos like 08;45 #557

Closed
kayg wants to merge 0 commits from issue/log-times-lenient into dev
pull from: issue/log-times-lenient
merge into: kayg:dev
Conversation 1 Commits - Files changed -
kayg commented 2026-10-17 17:52:35 +00:00
Owner
Copy link

Closes #556

Bug: 08;45 - 11:30 Getting ready (semicolon typo) was plain text, the 11:30 too, while 08:15 - 08:45 was grey.

Fix (lenient logs, r5): hour/minute separator : . ; h, 1-2 digit hours, - – — to between times (spaces optional), optional am/pm. Bare 0845 stays text. One reader: the grey in the editor now uses the same parser as + Log (logTimeSpan), and the Rust twin gets the same rules, both on contracts/log-entries.json. Text on disk is never rewritten.

Open question for kayg: should we normalise the stored text (08;45 to 08:45)? Left as is.

Closes #556 Bug: `08;45 - 11:30 Getting ready` (semicolon typo) was plain text, the 11:30 too, while `08:15 - 08:45` was grey. Fix (lenient logs, r5): hour/minute separator `: . ; h`, 1-2 digit hours, `- – — to` between times (spaces optional), optional am/pm. Bare `0845` stays text. One reader: the grey in the editor now uses the same parser as + Log (`logTimeSpan`), and the Rust twin gets the same rules, both on `contracts/log-entries.json`. Text on disk is never rewritten. Open question for kayg: should we normalise the stored text (08;45 to 08:45)? Left as is.
Log times: forgive typos like 08;45 (#556)
Some checks failed
CI / rust (pull_request) Failing after 8s
Details
CI / server (pull_request) Successful in 25s
Details
CI / web (pull_request) Failing after 28s
Details
4b4d38256b 
Accept : . ; h between hour and minute, 1-2 digit hours, - – — or 'to'
between times, optional am/pm. The editor's grey now uses the same reader
as + Log; the Rust twin and shared vectors agree. Text is never rewritten.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
kayg commented 2026-10-17 20:10:22 +00:00
Author
Owner
Copy link

Landed in train y, #565.

Landed in train y, #565.
kayg closed this pull request 2026-10-17 20:10:23 +00:00
Some checks failed
CI / rust (pull_request) Failing after 8s
Details
CI / server (pull_request) Successful in 25s
Details
CI / web (pull_request) Failing after 28s
Details

Pull request closed

This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
backend
Rust server

  
design
Mockups, UI and TASTE

  
docs
ADRs, CONTEXT, TASTE

  
idea
Parked idea, never worked until promoted

  
infra
Deploy, CI, VMs

  
perf
Performance and budgets

  
reliability
Data integrity, corruption traps

  
web
Web client

No labels
backend
design
docs
idea
infra
perf
reliability
web
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
kayg/calternal!557
No description provided.