Skip to content

feat(notifications): display who's the author#3029

Closed
yuriteixeira wants to merge 1 commit into
gitify-app:mainfrom
yuriteixeira:feat/display-notification-author
Closed

feat(notifications): display who's the author#3029
yuriteixeira wants to merge 1 commit into
gitify-app:mainfrom
yuriteixeira:feat/display-notification-author

Conversation

@yuriteixeira

@yuriteixeira yuriteixeira commented Jul 8, 2026

Copy link
Copy Markdown

Summary

  • Include issue and pull request authors in the notification reason title text
  • Keep author names out of the visible footer layout to avoid crowding row actions
  • Update notification footer tests and snapshots

Validation

  • pnpm exec vp test --changed --passWithNoTests --update
  • pnpm check

@github-actions github-actions Bot added the enhancement New feature or enhancement to existing functionality label Jul 8, 2026
@setchy

setchy commented Jul 8, 2026

Copy link
Copy Markdown
Member

I don't think we have enough horizontal screen real estate to show notification author handles, as that'll truncate our notification metric pills.

@setchy

setchy commented Jul 8, 2026

Copy link
Copy Markdown
Member

Do you have some screenshots of what this feature will look like?

@yuriteixeira

yuriteixeira commented Jul 9, 2026

Copy link
Copy Markdown
Author

Fair point, I will look into a way to better display this and will get back to you.

- Display the author in the notification footers for issues and pull requests.
- Link author names to their GitHub profiles.

Made with ♥️ with the help of spotify-hendrix/gpt-5.5 & https://pi.dev
@yuriteixeira yuriteixeira force-pushed the feat/display-notification-author branch from d7dae1e to da02c1e Compare July 9, 2026 12:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or enhancement to existing functionality

Development

Successfully merging this pull request may close these issues.

2 participants