aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorYehonatan Ezron <37303618+jonatan5524@users.noreply.github.com>2022-05-28 23:44:51 +0300
committerGitHub <noreply@github.com>2022-05-28 16:44:51 -0400
commitf2439b7e4c47de018e1678ceaa434e4b3d734a83 (patch)
tree6188e2814a7131b407d1f3efb2e3bde97c70a149
parent65e0688a5c9dacad50e71024b7529fdf0e3c2e9c (diff)
downloadgitea-f2439b7e4c47de018e1678ceaa434e4b3d734a83.tar.gz
gitea-f2439b7e4c47de018e1678ceaa434e4b3d734a83.zip
fix(telegram): fix link bot to release tag (#19830)
* fix(telegram): fix link bot to release tag * test(webhook): fix matrixPayload Release * test(webhook): fix TestTelegramPayload Release
-rw-r--r--services/webhook/general.go2
-rw-r--r--services/webhook/matrix_test.go4
-rw-r--r--services/webhook/slack_test.go2
-rw-r--r--services/webhook/telegram_test.go2
4 files changed, 5 insertions, 5 deletions
diff --git a/services/webhook/general.go b/services/webhook/general.go
index 5080cf98dc..e8006fabae 100644
--- a/services/webhook/general.go
+++ b/services/webhook/general.go
@@ -141,7 +141,7 @@ func getPullRequestPayloadInfo(p *api.PullRequestPayload, linkFormatter linkForm
func getReleasePayloadInfo(p *api.ReleasePayload, linkFormatter linkFormatter, withSender bool) (text string, color int) {
repoLink := linkFormatter(p.Repository.HTMLURL, p.Repository.FullName)
- refLink := linkFormatter(p.Repository.HTMLURL+"/src/"+util.PathEscapeSegments(p.Release.TagName), p.Release.TagName)
+ refLink := linkFormatter(p.Repository.HTMLURL+"/releases/tag/"+util.PathEscapeSegments(p.Release.TagName), p.Release.TagName)
switch p.Action {
case api.HookReleasePublished:
diff --git a/services/webhook/matrix_test.go b/services/webhook/matrix_test.go
index 3cc7c7518f..34196aedf0 100644
--- a/services/webhook/matrix_test.go
+++ b/services/webhook/matrix_test.go
@@ -165,8 +165,8 @@ func TestMatrixPayload(t *testing.T) {
require.NotNil(t, pl)
require.IsType(t, &MatrixPayloadUnsafe{}, pl)
- assert.Equal(t, "[[test/repo](http://localhost:3000/test/repo)] Release created: [v1.0](http://localhost:3000/test/repo/src/v1.0) by [user1](https://try.gitea.io/user1)", pl.(*MatrixPayloadUnsafe).Body)
- assert.Equal(t, `[<a href="http://localhost:3000/test/repo">test/repo</a>] Release created: <a href="http://localhost:3000/test/repo/src/v1.0">v1.0</a> by <a href="https://try.gitea.io/user1">user1</a>`, pl.(*MatrixPayloadUnsafe).FormattedBody)
+ assert.Equal(t, "[[test/repo](http://localhost:3000/test/repo)] Release created: [v1.0](http://localhost:3000/test/repo/releases/tag/v1.0) by [user1](https://try.gitea.io/user1)", pl.(*MatrixPayloadUnsafe).Body)
+ assert.Equal(t, `[<a href="http://localhost:3000/test/repo">test/repo</a>] Release created: <a href="http://localhost:3000/test/repo/releases/tag/v1.0">v1.0</a> by <a href="https://try.gitea.io/user1">user1</a>`, pl.(*MatrixPayloadUnsafe).FormattedBody)
})
}
diff --git a/services/webhook/slack_test.go b/services/webhook/slack_test.go
index 1fa7777328..8278afb69a 100644
--- a/services/webhook/slack_test.go
+++ b/services/webhook/slack_test.go
@@ -154,7 +154,7 @@ func TestSlackPayload(t *testing.T) {
require.NotNil(t, pl)
require.IsType(t, &SlackPayload{}, pl)
- assert.Equal(t, "[<http://localhost:3000/test/repo|test/repo>] Release created: <http://localhost:3000/test/repo/src/v1.0|v1.0> by <https://try.gitea.io/user1|user1>", pl.(*SlackPayload).Text)
+ assert.Equal(t, "[<http://localhost:3000/test/repo|test/repo>] Release created: <http://localhost:3000/test/repo/releases/tag/v1.0|v1.0> by <https://try.gitea.io/user1|user1>", pl.(*SlackPayload).Text)
})
}
diff --git a/services/webhook/telegram_test.go b/services/webhook/telegram_test.go
index 6a3682847c..2f83090166 100644
--- a/services/webhook/telegram_test.go
+++ b/services/webhook/telegram_test.go
@@ -154,7 +154,7 @@ func TestTelegramPayload(t *testing.T) {
require.NotNil(t, pl)
require.IsType(t, &TelegramPayload{}, pl)
- assert.Equal(t, `[<a href="http://localhost:3000/test/repo">test/repo</a>] Release created: <a href="http://localhost:3000/test/repo/src/v1.0">v1.0</a> by <a href="https://try.gitea.io/user1">user1</a>`, pl.(*TelegramPayload).Message)
+ assert.Equal(t, `[<a href="http://localhost:3000/test/repo">test/repo</a>] Release created: <a href="http://localhost:3000/test/repo/releases/tag/v1.0">v1.0</a> by <a href="https://try.gitea.io/user1">user1</a>`, pl.(*TelegramPayload).Message)
})
}