Skip to content

Correct issue and PR notification rendering - #17

Merged
hnrobert merged 1 commit into
hnrobert:developfrom
Nanaloveyuki:fix/issue-11-message-rendering
Jul 20, 2026
Merged

Correct issue and PR notification rendering#17
hnrobert merged 1 commit into
hnrobert:developfrom
Nanaloveyuki:fix/issue-11-message-rendering

Conversation

@Nanaloveyuki

@Nanaloveyuki Nanaloveyuki commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Populate the common action template field for all webhook event types.
  • Link Issue comments to their parent Issue and fix the malformed PR update emoji.
  • Remove the redundant PR separator that appeared when the PR body was empty.

Validation

  • go test ./internal/handler ./internal/template ./internal/confign- go build ./...n
    Related to Message displaying bugs discovered during daily use #11. This intentionally does not cover image and code-block rendering, which requires a separate Feishu card/content conversion design.

@hnrobert

Copy link
Copy Markdown
Owner

sugoi

@hnrobert hnrobert linked an issue Jul 20, 2026 that may be closed by this pull request
@hnrobert hnrobert changed the title fix: correct issue and PR notification rendering Correct issue and PR notification rendering Jul 20, 2026
@hnrobert
hnrobert merged commit d14dea2 into hnrobert:develop Jul 20, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Message displaying bugs discovered during daily use

2 participants