-
-
Notifications
You must be signed in to change notification settings - Fork 19
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
[FEATURE] Adding an optional 'date' & 'hours' field to the 'zeit export' command. #32
Comments
Sounds good to me! In order to not break people's scripts, making the output optional is a good idea.
What do you think? |
Sure, I don´t mind them being long flags instead. I have it implemented so that I just extend the If so I will get to it right away :) |
One question that I do have. Should I also adapt the 'tyme' export? I'm not sure how that app/program functions and I don´t know if it can handle extra fields etc. Let me know :) |
Nope, I don't think the Let me get back to the other things in your PR. |
Hey, just something I've been implementing in my personal varation of this as I find it quite usefull is to have two additional fields in the 'export' JSON that comes out of
zeit export
.Now I think since maybe not everyone wants these it would be best to make them 'optional' so that you have to enable them.
Some questions for that, that I would have then would be.
Let me know if this is something that might be interesting, and I can make a PR for it in time.
And also, any additional requierements or things it should do.
Best
The text was updated successfully, but these errors were encountered: