Fix workflow trigger event IssueChangeXXX bug (#29559)
Bugs from #29308 Follow #29467 partly fix #29558 (cherry picked from commit 9616dbec334aacb32c6d73b01fd749b11b1e3cdb)
This commit is contained in:
parent
c69a0bb506
commit
af19bae813
1 changed files with 14 additions and 2 deletions
|
@ -171,14 +171,26 @@ func (n *actionsNotifier) IssueChangeMilestone(ctx context.Context, doer *user_m
|
||||||
} else {
|
} else {
|
||||||
action = api.HookIssueDemilestoned
|
action = api.HookIssueDemilestoned
|
||||||
}
|
}
|
||||||
notifyIssueChange(ctx, doer, issue, webhook_module.HookEventPullRequestMilestone, action)
|
|
||||||
|
hookEvent := webhook_module.HookEventIssueMilestone
|
||||||
|
if issue.IsPull {
|
||||||
|
hookEvent = webhook_module.HookEventPullRequestMilestone
|
||||||
|
}
|
||||||
|
|
||||||
|
notifyIssueChange(ctx, doer, issue, hookEvent, action)
|
||||||
}
|
}
|
||||||
|
|
||||||
func (n *actionsNotifier) IssueChangeLabels(ctx context.Context, doer *user_model.User, issue *issues_model.Issue,
|
func (n *actionsNotifier) IssueChangeLabels(ctx context.Context, doer *user_model.User, issue *issues_model.Issue,
|
||||||
_, _ []*issues_model.Label,
|
_, _ []*issues_model.Label,
|
||||||
) {
|
) {
|
||||||
ctx = withMethod(ctx, "IssueChangeLabels")
|
ctx = withMethod(ctx, "IssueChangeLabels")
|
||||||
notifyIssueChange(ctx, doer, issue, webhook_module.HookEventPullRequestLabel, api.HookIssueLabelUpdated)
|
|
||||||
|
hookEvent := webhook_module.HookEventIssueLabel
|
||||||
|
if issue.IsPull {
|
||||||
|
hookEvent = webhook_module.HookEventPullRequestLabel
|
||||||
|
}
|
||||||
|
|
||||||
|
notifyIssueChange(ctx, doer, issue, hookEvent, api.HookIssueLabelUpdated)
|
||||||
}
|
}
|
||||||
|
|
||||||
func notifyIssueChange(ctx context.Context, doer *user_model.User, issue *issues_model.Issue, event webhook_module.HookEventType, action api.HookIssueAction) {
|
func notifyIssueChange(ctx context.Context, doer *user_model.User, issue *issues_model.Issue, event webhook_module.HookEventType, action api.HookIssueAction) {
|
||||||
|
|
Loading…
Reference in a new issue