Skip to content

refactor: use display names in insights section#110

Merged
O2sa merged 1 commit into
O2sa:mainfrom
Ananta2003:refactor/insights-name-field
Apr 24, 2026
Merged

refactor: use display names in insights section#110
O2sa merged 1 commit into
O2sa:mainfrom
Ananta2003:refactor/insights-name-field

Conversation

@Ananta2003

Copy link
Copy Markdown
Contributor

✅ Changes Completed

  • Updated getInsights logic to use user.name instead of user.username
  • Added fallback to username when display name is unavailable
  • Improved user-facing insight text readability

Closes #100

@vercel

vercel Bot commented Apr 24, 2026

Copy link
Copy Markdown

Someone is attempting to deploy a commit to the osama's projects Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions

Copy link
Copy Markdown

Thank you for the pull request! ✅

A maintainer will review this soon. Please be patient while we take a look. 🙌

@vercel

vercel Bot commented Apr 24, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
dev-impact Ready Ready Preview, Comment Apr 24, 2026 6:09pm

@O2sa O2sa merged commit 3f89eb4 into O2sa:main Apr 24, 2026
3 checks passed
@github-actions

Copy link
Copy Markdown

Thank you, @Ananta2003! Another great contribution merged! 🚀

You've been a fantastic contributor! We truly appreciate your continued support.

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.

refactor: Use 'name' instead of 'username' in insights section

2 participants