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

[WIP] Issue webhook #908

Closed
wants to merge 10 commits into from
Closed

[WIP] Issue webhook #908

wants to merge 10 commits into from

Conversation

thehowl
Copy link
Contributor

@thehowl thehowl commented Feb 11, 2017

Depends on go-gitea/go-sdk#38

Will resolve #132 and #895 partially (still needs issue/pr comments, which will be done in another PR, following the GitHub webhooks which separate between comments and other PR/issue events).

Currently only needs Slack webhooks, then we're good to go.

@lunny lunny modified the milestones: 1.1.0, 1.x.x Feb 12, 2017
@lunny lunny added the type/feature Completely new functionality. Can only be merged if feature freeze is not active. label Feb 12, 2017
@lunny lunny modified the milestones: 1.2.0, 1.x.x Feb 23, 2017
@lunny
Copy link
Member

lunny commented Feb 23, 2017

conflicted

@tboerger tboerger added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Feb 23, 2017
@lunny lunny mentioned this pull request Feb 23, 2017
20 tasks
@lunny
Copy link
Member

lunny commented Apr 6, 2017

@thehowl any update?

@thehowl
Copy link
Contributor Author

thehowl commented Apr 6, 2017

>_> Been busy as of lately so wasn't able to complete this, I'm hoping to finish this sometime soon! 😥

@lunny lunny modified the milestones: 1.x.x, 1.2.0 Apr 20, 2017
@hhenkel
Copy link
Contributor

hhenkel commented Jul 3, 2017

@thehowl any news on that? Would be rather good to have that, as we then could integrate our PR workflow even more into our chat.

@thehowl
Copy link
Contributor Author

thehowl commented Jul 7, 2017

I'm really sorry, but I am really filled with stuff to do that I can't finish it. If one of the maintainers wants to close the PR or want to make the edits still needed on this one, they're very welcome to do so. Again, I'm very sorry for not being able to finish this 😢

@lunny lunny mentioned this pull request Aug 29, 2017
6 tasks
@thehowl
Copy link
Contributor Author

thehowl commented Oct 19, 2017

seeing as this PR was done a long time ago, I'm going to close this and try to work on this again on another branch

@thehowl thehowl closed this Oct 19, 2017
@lunny lunny removed this from the 1.x.x milestone Oct 20, 2017
@go-gitea go-gitea locked and limited conversation to collaborators Nov 23, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. type/feature Completely new functionality. Can only be merged if feature freeze is not active.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Webhooks for issues
4 participants