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

Migration to Toggl API v9 #83

Open
morvans opened this issue Apr 24, 2024 · 3 comments · May be fixed by #84
Open

Migration to Toggl API v9 #83

morvans opened this issue Apr 24, 2024 · 3 comments · May be fixed by #84

Comments

@morvans
Copy link

morvans commented Apr 24, 2024

Summary

Toggl will soon deprecate their v9 API.
See https://engineering.toggl.com/changes/2022/10/01/api-v8-deprecation/index.html

As far as I can tell, toggl2redmine only relies on the time entries fetching API endpoint, which is impacted by the introduced breaking changes.
See https://engineering.toggl.com/docs/additional/v8_migration_guide/index.html#time-entries

Searching for "v8" in the code base only yields minimal occurences.
See https://github.com/search?q=repo%3Ajigarius%2Ftoggl2redmine%20v8&type=code

To do

Replace calls to the deprecated API endpoint with the new v9 endpoint with the necessary adaptations.

@morvans morvans added the bug Something isn't working label Apr 24, 2024
@jigarius
Copy link
Owner

jigarius commented May 1, 2024

Hey! Thanks for reporting this issue. I'll try to take a look at this at the earliest. The calls should be toggl_service.rb.

That said, if you have the time, please feel free to create a pull request to help fix this issue.

@jigarius jigarius added Task Priority: High and removed bug Something isn't working labels May 1, 2024
@kdborg
Copy link

kdborg commented May 16, 2024

Hi @jigarius, I work at EW and I have a patch to push and Github is denying me access.

Here are the changes:
update.patch

@jigarius
Copy link
Owner

@kdborg thanks for the patch. I'll look at it this weekend.

@jigarius jigarius linked a pull request May 21, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants