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

Add option to edit time based on start and end time. #1

Open
akoppela opened this issue Sep 9, 2019 · 5 comments
Open

Add option to edit time based on start and end time. #1

akoppela opened this issue Sep 9, 2019 · 5 comments

Comments

@akoppela
Copy link
Contributor

akoppela commented Sep 9, 2019

Hi! Thanks for the package. At the moment it's only possible to edit amount oh hours for an entry, am I right? It would be nice to have an option to edit start and end time as well. Do you think it's something you'd like to add? I may push a PR if you want so.

@xendk
Copy link
Owner

xendk commented Sep 9, 2019

Hi, thanks.

Supporting range based tracking would be nice, but it might get a bit complicated. As we're using duration based tracking around here, I can't actually test that it works.

According to the time entry API docs it is dependent on the setup of the company in Harvest. Annoyingly I get access denied on the company API call that's supposed to tell which it is, so I couldn't auto-detect it. It would have to be a custom variable that's used to set it then.

Hopefully it's just having the time query and the create timer functions switch based on the variable, and the listing and other stuff should work as they just look at the hours key of the response, which I assume Harvest will keep updated.

Sure, give it a shot.

@akoppela
Copy link
Contributor Author

akoppela commented Sep 9, 2019

#4 is created.

@akoppela
Copy link
Contributor Author

akoppela commented Sep 9, 2019

The PR adds the simple solution. It's just one more function. But it should be definitely possible to have a custom variable to automate the process.

@akoppela
Copy link
Contributor Author

I updated the PR with a custom variable to switch between Harvest timer modes.

@akoppela
Copy link
Contributor Author

akoppela commented Jan 8, 2022

I'm ready to come back to this issue now.

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 a pull request may close this issue.

2 participants