Skip to content

fix(maps): include seconds in timezone UTC offset output - #16300

Merged
teknium1 merged 2 commits into
mainfrom
hermes/hermes-e6c174d2
Apr 27, 2026
Merged

fix(maps): include seconds in timezone UTC offset output#16300
teknium1 merged 2 commits into
mainfrom
hermes/hermes-e6c174d2

Conversation

@teknium1

Copy link
Copy Markdown
Contributor

Salvage of #15086 by @0z1-ghb onto current main. Authorship preserved via cherry-pick + rebase-merge.

Summary

Maps timezone command now includes seconds in the UTC offset string when timeapi.io reports a non-zero seconds field. Previously the currentUtcOffset path extracted seconds but discarded it, and the standardUtcOffset path didn't read it at all.

Changes

  • skills/productivity/maps/scripts/maps_client.py: append :SS to utc_offset when seconds != 0, on both currentUtcOffset and standardUtcOffset code paths.

Validation

offset_info output
hours=5, minutes=30, seconds=0 +05:30
hours=5, minutes=30, seconds=45 +05:30:45
hours=-8, minutes=0, seconds=0 -08:00

Closes #15086.

0z1-ghb added 2 commits April 26, 2026 19:00
fix: include seconds in timezone UTC offset output
@teknium1
teknium1 merged commit 419535f into main Apr 27, 2026
11 of 12 checks passed
@teknium1
teknium1 deleted the hermes/hermes-e6c174d2 branch April 27, 2026 02:03
@teknium1 teknium1 mentioned this pull request Apr 27, 2026
@alt-glitch alt-glitch added type/bug Something isn't working P3 Low — cosmetic, nice to have tool/skills Skills system (list, view, manage) labels Apr 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

P3 Low — cosmetic, nice to have tool/skills Skills system (list, view, manage) type/bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants