search

sephii / taxi

Taxi is a CLI to time tracking backends
https://taxi-timesheets.readthedocs.io/
Do What The F*ck You Want To Public License
41 stars 15 forks source link

Aliases are forced as lower case and projects identifiers to integers #129

Closed gnutix closed 4 years ago

gnutix commented 4 years ago

@sephii With the taxi-tempo plugin, we can use JIRA tickets as aliases, like INFRA-5 or OPS-318. However, because of how taxi currently works, we're forced to write them in lower case in our timesheet files.

In the same type of issues, the project identifier is forced to be an integer, where I would have used the first part of the JIRA issue ("INFRA"), which lead us to do quite a hack to convert it back and forth and therefore the aliases generated in ~/.taxirc are obfuscated (where they wouldn't need to be).

Any chance these constraints might be made optional / removed ?

sephii commented 4 years ago

Can you describe what's happening when you use a non-lowercase alias? I don't remember any constraint regarding aliases but it's been a long time since I last worked on the parser. ^^

For the project id constraint I wouldn't see a problem accepting an str type. I'd be happy to review a PR, or this will have to wait a bit until I find some time to work on this.

gnutix commented 4 years ago

I proposed a PR! The issue with uppercase aliases in timesheet entries was that it resulted in an "inexistent alias" warning and the entry was skipped.