Skip to content

Add profile timezone fallback for leave creation#11

Merged
jernejk merged 1 commit into
mainfrom
codex/issue-10-leave-timezone-resilience
Jun 30, 2026
Merged

Add profile timezone fallback for leave creation#11
jernejk merged 1 commit into
mainfrom
codex/issue-10-leave-timezone-resilience

Conversation

@jernejk

@jernejk jernejk commented Jun 30, 2026

Copy link
Copy Markdown
Member

Summary

  • Fixes Bug - Error 500 while user does not have any timezone set #10 by making leave creation resolve timezone in this order: explicit override, TimePro user profile timezone, then CLI/MCP host machine timezone.
  • Adds tp leave create --timezone <TIMEZONE_ID> and MCP create_leave timeZoneId support for agent-controlled timezone overrides.
  • Keeps the machine timezone fallback for profiles without timezone data, so the fallback is controlled by the runtime environment rather than automatic browser state.
  • Adds an MCP create_leave path that shares the same parsing and timezone behavior as the CLI.

Sanity check

  • Azure DevOps PR 3427 is completed/merged server-side, and staging no longer returns the previous null-timezone 500 in the duplicate-create smoke check.
  • This PR keeps the tools side resilient rather than relying on the server alone.

Validation

  • dotnet build --no-restore
  • dotnet test --no-build
  • dotnet run --no-build --project src/SSW.TimePro.Cli -- leave create --help
  • git diff --check

@jernejk jernejk force-pushed the codex/issue-10-leave-timezone-resilience branch from 5eaa166 to e91484e Compare June 30, 2026 11:45
@jernejk jernejk marked this pull request as ready for review June 30, 2026 11:49
@jernejk jernejk merged commit b131dbc into main Jun 30, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bug - Error 500 while user does not have any timezone set

1 participant