search

lucassabreu / clockify-cli

A simple cli to manage your time entries on Clockify from terminal
https://clockify-cli.netlify.app/
Apache License 2.0
167 stars 24 forks source link

feat: Support for times without leading '0' #251

Closed aVolpe closed 12 months ago

aVolpe commented 12 months ago

The web interface of clockify supports times in format HMM (1 digit for the hour and 2 digits for the minutes), for example 915 is 09:15.

I am not very familiar with go, but I added a test (following another test in this repo) to make sure everything works.

To check the new test usego test -v pkg/timehlp/time_test.go.

The parser now also supports the format H:MM also is supported by the web interface, to keep the names short the suffix 'WL' was used (without leading 0, suggestions welcome!).

aVolpe commented 12 months ago

thanks @lucassabreu , nothing was changed nor reported by that command:

image

(The first version of this pr has some lint errors that were reported by DeepSource, but those were fixed in a force push).

lucassabreu commented 12 months ago

i guess i saw it before the force-push.

lucassabreu commented 12 months ago

thank you for the contribution, is at the release v0.46.0 now

aVolpe commented 12 months ago

thank you for the contribution, is at the release v0.46.0 now

I downloaded the version through homebrew and everything works! Thanks for the CLI!