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 selfhosted wakatime integration (wakapi) #276

Merged
merged 5 commits into from
Apr 28, 2021

Conversation

MeerBiene
Copy link
Contributor

This PR adds the option to use a selfhosted wakatime instance for the wakatime plugin.

I tested it against this project called Wakapi which mimics the same endpoints as wakatime does, so its just a matter of using a different api url

Copy link
Owner

@lowlighter lowlighter left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks a lot for your contribution 👍 !
I requested some changes but nothing major 🙂

source/plugins/wakatime/metadata.yml Outdated Show resolved Hide resolved
source/plugins/wakatime/index.mjs Outdated Show resolved Hide resolved
source/plugins/wakatime/index.mjs Outdated Show resolved Hide resolved
source/plugins/wakatime/metadata.yml Outdated Show resolved Hide resolved
source/plugins/wakatime/metadata.yml Outdated Show resolved Hide resolved
@MeerBiene
Copy link
Contributor Author

I think I got everything resolved, let me know if otherwise :)

Copy link
Owner

@lowlighter lowlighter left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Almost!
Since plugin_wakatime_apiurl has been renamed to plugin_wakatime_url the inputs parser will send back url instead of apiurl 🙂

After this it should be ok 👍
(builds are currently broke because I'm experimenting stuff with vercel so it's ok)

source/plugins/wakatime/index.mjs Outdated Show resolved Hide resolved
source/plugins/wakatime/index.mjs Outdated Show resolved Hide resolved
@lowlighter lowlighter merged commit eda35ca into lowlighter:master Apr 28, 2021
@lowlighter
Copy link
Owner

Thanks!

Repository owner deleted a comment from vercel bot Apr 28, 2021
Repository owner deleted a comment from vercel bot Apr 28, 2021
@github-actions github-actions bot locked and limited conversation to collaborators May 13, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants