-
Notifications
You must be signed in to change notification settings - Fork 16
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
Support for "duronly" entries. #9
Comments
When designing this plugin, I tried to make the logic as clear and simple as possible. The entries are imported once and only once. Ever since that moment they become independent of each other. This approach gives you the following benefits:
Unfortunately this way of dealing with things makes it conceptually impossible to use pause-and-resume Toggl entries, which I too favored before starting to use this plugin. If you have ideas of another concise algorithm that will allow you to exercise pause-and-resume stuff, I would happily implement it. |
I see. I'd propose to always update the time-entries in redmine:
I think 3. is not such a big issue since users who log time in toggl also tend to edit those entries there. Also editing existing time-entries in redmine is not that common (adding time entries is). |
What about deletion in Redmine or in Toggl? Or changing the entries in such a way, that they become disconnected from the tasks or reconnected to different ones? |
Good points.
Also to avoid syncing "incomplete" time entries the import could be limited to all time-entries started at least X hours in the past. |
Toggl has a setting that allows users to pause and resume time entries. Those time-entries then don't get a "stop"-time. Due to the filter in this plugin those time entries are ignored even though they are completed.
Currently I'm using a condition like
((!entry["stop"].nil? && !entry["stop"].empty?) || (entry["duronly"] && !entry["duration"].empty?))
to get all the time entries, but I'm unsure whether this is the best approach.The text was updated successfully, but these errors were encountered: