Skip to content

Commit 7cccada

Browse files
danielfliralunny
authored andcommitted
Fix assets key on release webhook (#8253)
* Fix assets key on release webhook * Fix assets key on release webhook * Add message to log on AddReleaseAttachments failure
1 parent 5a438ee commit 7cccada

File tree

1 file changed

+4
-2
lines changed

1 file changed

+4
-2
lines changed

services/release/release.go

+4-2
Original file line numberDiff line numberDiff line change
@@ -109,12 +109,14 @@ func UpdateRelease(doer *models.User, gitRepo *git.Repository, rel *models.Relea
109109
return err
110110
}
111111

112+
if err = models.AddReleaseAttachments(rel.ID, attachmentUUIDs); err != nil {
113+
log.Error("AddReleaseAttachments: %v", err)
114+
}
115+
112116
if err = rel.LoadAttributes(); err != nil {
113117
return err
114118
}
115119

116-
err = models.AddReleaseAttachments(rel.ID, attachmentUUIDs)
117-
118120
// even if attachments added failed, hooks will be still triggered
119121
mode, _ := models.AccessLevel(doer, rel.Repo)
120122
if err1 := models.PrepareWebhooks(rel.Repo, models.HookEventRelease, &api.ReleasePayload{

0 commit comments

Comments
 (0)