Skip to content

fix: use orange header for manual execution notifications#4745

Open
huanghongbo-hhb wants to merge 9 commits into
koderover:mainfrom
huanghongbo-hhb:fix/manual-exec-feishu-orange-title-v2
Open

fix: use orange header for manual execution notifications#4745
huanghongbo-hhb wants to merge 9 commits into
koderover:mainfrom
huanghongbo-hhb:fix/manual-exec-feishu-orange-title-v2

Conversation

@huanghongbo-hhb

@huanghongbo-hhb huanghongbo-hhb commented Jun 4, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Update Feishu pause notifications to use an orange header and show Waiting for Manual Execution.

Main Changes

  • Apply the orange header template when workflow status is pause in Feishu notifications.
  • Override Feishu pause display text to Waiting for Manual Execution.
  • Skip the generic feishu_person pause notification when a manual-execution pending notification will also be sent, to avoid duplicate Feishu cards.

Test

  • env GOCACHE=/private/tmp/zadig-gocache go test -vet=off ./pkg/microservice/aslan/core/common/service/instantmessage ./pkg/microservice/aslan/core/common/service/workflowcontroller ./pkg/microservice/aslan/core/workflow/service/workflow/controller ./pkg/microservice/aslan/core/workflow/service/workflow

Contact


This change is Reviewable

Signed-off-by: huanghongbo-hhb <huanghongbo@koderover.com>
Signed-off-by: huanghongbo-hhb <huanghongbo@koderover.com>
Signed-off-by: huanghongbo-hhb <huanghongbo@koderover.com>
Signed-off-by: huanghongbo-hhb <huanghongbo@koderover.com>
Signed-off-by: huanghongbo-hhb <huanghongbo@koderover.com>
Comment thread pkg/microservice/aslan/core/common/service/instantmessage/workflow_task.go Outdated
Signed-off-by: huanghongbo-hhb <huanghongbo@koderover.com>
Signed-off-by: huanghongbo-hhb <huanghongbo@koderover.com>
Signed-off-by: huanghongbo-hhb <huanghongbo@koderover.com>
Signed-off-by: huanghongbo-hhb <huanghongbo@koderover.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants