starred/pi-skills
1
0
Fork 0
mirror of https://github.com/badlogic/pi-skills.git synced 2026-04-27 04:55:58 +03:00
Code Issues 13 Projects Releases Packages Wiki Activity

[PR #14] fix(gccli): correct datetime format documentation #16

New issue
Closed
opened 2026-03-07 13:58:01 +03:00 by kerem · 0 comments
kerem commented 2026-03-07 13:58:01 +03:00
Owner
Copy link

Original Pull Request: https://github.com/badlogic/pi-skills/pull/14

State: closed
Merged: No

Problem

The gccli skill documentation incorrectly stated that local time format (YYYY-MM-DDTHH:MM:SS) without timezone works. This caused confusion and failed API calls.

Testing Results

Format Query (--from/--to) Create (--start/--end)
YYYY-MM-DD ❌ Bad Request ❌ (unless --all-day)
YYYY-MM-DDTHH:MM:SS (no TZ) ❌ Bad Request ❌ Missing time zone
YYYY-MM-DDTHH:MM:SSZ (UTC) ✅ Works ✅ Works
YYYY-MM-DDTHH:MM:SS-08:00 (offset) ✅ Works ✅ Works
YYYY-MM-DD + --all-day N/A ✅ Works

Fix

Updated the Date/Time Format section to:

  • Clarify that all datetime parameters (--from, --to, --start, --end) require a timezone
  • List valid formats: UTC (Z suffix) or explicit offset (±HH:MM)
  • Add warning that local time without timezone does NOT work
  • Clarify YYYY-MM-DD only works with --all-day flag for creating events

Verification

Tested in a fresh session after the fix — works flawlessly.

**Original Pull Request:** https://github.com/badlogic/pi-skills/pull/14 **State:** closed **Merged:** No --- ## Problem The gccli skill documentation incorrectly stated that local time format (`YYYY-MM-DDTHH:MM:SS`) without timezone works. This caused confusion and failed API calls. ## Testing Results | Format | Query (`--from/--to`) | Create (`--start/--end`) | |--------|----------------------|--------------------------| | `YYYY-MM-DD` | ❌ Bad Request | ❌ (unless `--all-day`) | | `YYYY-MM-DDTHH:MM:SS` (no TZ) | ❌ Bad Request | ❌ Missing time zone | | `YYYY-MM-DDTHH:MM:SSZ` (UTC) | ✅ Works | ✅ Works | | `YYYY-MM-DDTHH:MM:SS-08:00` (offset) | ✅ Works | ✅ Works | | `YYYY-MM-DD` + `--all-day` | N/A | ✅ Works | ## Fix Updated the Date/Time Format section to: - Clarify that all datetime parameters (`--from`, `--to`, `--start`, `--end`) require a timezone - List valid formats: UTC (`Z` suffix) or explicit offset (`±HH:MM`) - Add warning that local time without timezone does NOT work - Clarify `YYYY-MM-DD` only works with `--all-day` flag for creating events ## Verification Tested in a fresh session after the fix — works flawlessly.
kerem 2026-03-07 13:58:01 +03:00
kerem changed title from [PR #14] [CLOSED] fix(gccli): correct datetime format documentation to [PR #14] fix(gccli): correct datetime format documentation 2026-03-07 19:42:46 +03:00
Sign in to join this conversation.
No Branch/Tag specified
Branches Tags
main
No results found.
Labels
Clear labels   
pull-request

Mirrored from GitHub Pull Request

No labels
pull-request
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
starred/pi-skills#16
No description provided.