Skip to content

feat: add URL links to all GitHub event notifications#7

Open
IYENTeam wants to merge 1 commit into
mainfrom
feat/github-event-urls
Open

feat: add URL links to all GitHub event notifications#7
IYENTeam wants to merge 1 commit into
mainfrom
feat/github-event-urls

Conversation

@IYENTeam
Copy link
Copy Markdown
Owner

Summary

Add clickable URL links to all GitHub event notifications that were missing them.

Problem

PR status-changed and issue events (opened/commented/closed) were rendered without links, making it impossible to click through to the actual GitHub page from Discord notifications. CI events already had URLs.

Changes

File What
src/source/github.rs Add url field to IssueSnapshot, html_url to GitHubIssue struct, pass through to events
src/events.rs Add url: String parameter to github_issue_opened/commented/closed constructors
src/render/default.rs Append URL to PR and issue event messages in Compact/Alert/Inline formats
src/daemon.rs Extract /issue/html_url from webhook payload
src/cli.rs Add --url flag to IssueOpened CLI command
src/main.rs Pass url through CLI dispatch
src/router.rs Update test fixtures

Before/After

# Before
PR IYENTeam/Hent-ai#10 open -> merged: docs: add onboarding section to READMEs

# After
PR IYENTeam/Hent-ai#10 open -> merged: docs: add onboarding section to READMEs · https://github.com/IYENTeam/Hent-ai/pull/10

Verification

  • cargo check: ✅ clean
  • cargo test: ✅ all passed
  • CI events: already had URLs, no change needed

- Add url field to IssueSnapshot and GitHubIssue structs
- Pass html_url through issue-opened/commented/closed events
- Render URL in PR status-changed (Compact/Alert/Inline formats)
- Render URL in issue-opened/commented/closed (all formats)
- CI events already included URLs, no change needed
- Update CLI IssueOpened command to accept --url flag
- Update webhook handler to extract issue html_url
- Fix all tests for new url parameter
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.

1 participant