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

Update Toggl endpoint. #8

Closed
wants to merge 1 commit into from
Closed

Conversation

chrisgre
Copy link
Contributor

Toggl is updating the base url of their api to api.track.toggl.com, and is turning off traffic to the top level domain June 30th. This should take care of that switch.

Description of change

Toggl sunsetting top level domain traffic. Switching to api.track.toggl.com

Manual QA steps

  • ensure tap is pulling from the new url

Risks

  • Toggle is shutting off traffic to tld June 30.

Rollback steps

  • revert this branch

Toggl is updating the base url of their api to `api.track.toggl.com`, and is turning off traffic to the top level domain June 30th. This should take care of that switch.
@cmerrick
Copy link
Contributor

Hi @chrisgre, thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes.

@cmerrick
Copy link
Contributor

You did it @chrisgre!

Thank you for signing the Singer Contribution License Agreement.

This was referenced Jun 25, 2021
@luandy64
Copy link
Contributor

@chrisgre Thanks for working on this. I got this merged in #9 so I'm closing this PR

@luandy64 luandy64 closed this Jun 25, 2021
@luandy64 luandy64 mentioned this pull request Jun 25, 2021
@chrisgre
Copy link
Contributor Author

Cheers Andy, happy to help.

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 this pull request may close these issues.

3 participants