Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Determine date format to use based on system locale's in "When" inputs #762

Merged
merged 2 commits into from
Sep 17, 2024

Conversation

dbarnett
Copy link
Collaborator

@dbarnett dbarnett commented Sep 17, 2024

Fixes #542.
Fixes #143.

Note: This adds a new dependency on babel, which isn't tiny. COULD convert that to optional-dependencies if absolutely needed, but I'd prefer to keep it an unconditional dep since i18n/l10n are pretty important aspects of a tool like gcalcli.

@dbarnett
Copy link
Collaborator Author

I'm also considering to add a manual override for this in config and/or more in-app hints about how to configure $LC_TIME etc for users not seeing the results they want, but I haven't done that yet in this PR. Seems nontrivial to get that right...

@dbarnett dbarnett merged commit 093e6da into main Sep 17, 2024
10 checks passed
@dbarnett dbarnett deleted the datefmt branch September 17, 2024 18:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant