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

fix url in discord webhook #4951

Merged
merged 1 commit into from
Sep 18, 2018
Merged

fix url in discord webhook #4951

merged 1 commit into from
Sep 18, 2018

Conversation

esno
Copy link
Contributor

@esno esno commented Sep 17, 2018

opening issues generates a webhook to discord that contains
a url to the gitea api. the message title in discord is therefore
referencing to the api instead of the issue itself.

opening issues generates a webhook to discord that contains
a url to the gitea api. the message title in discord is therefore
referencing to the api instead of the issue itself.
@appleboy appleboy added this to the 1.6.0 milestone Sep 18, 2018
@bkcsoft bkcsoft added the lgtm/need 1 This PR needs approval from one additional maintainer to be merged. label Sep 18, 2018
@bkcsoft bkcsoft added lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. and removed lgtm/need 1 This PR needs approval from one additional maintainer to be merged. labels Sep 18, 2018
@lafriks lafriks merged commit 4176e33 into go-gitea:release/v1.5 Sep 18, 2018
@lafriks
Copy link
Member

lafriks commented Sep 18, 2018

Please send backport to release/v1.5 branch

@lafriks
Copy link
Member

lafriks commented Sep 18, 2018

Oh I just noticed that this PR is for release/v1.5 branch. You should send PR to master branch first next time. Please backport to master than.

@lafriks lafriks modified the milestones: 1.6.0, 1.5.2 Sep 18, 2018
@esno esno mentioned this pull request Sep 18, 2018
@esno
Copy link
Contributor Author

esno commented Sep 18, 2018

was late yesterday. sorry for that

@adelowo
Copy link
Member

adelowo commented Sep 18, 2018

Is this a fix for #4656 ?

@lunny
Copy link
Member

lunny commented Sep 18, 2018

@adelowo yes

@esno
Copy link
Contributor Author

esno commented Sep 18, 2018

thx @adelowo for pointing out that issue number. that's exactly the issue that is addressed by this PR.

@go-gitea go-gitea locked and limited conversation to collaborators Nov 24, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. type/bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants