[PR #1656] [MERGED] fix: use global_name for discord #3926

Closed
opened 2026-03-13 11:22:51 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/better-auth/better-auth/pull/1656
Author: @juliesaia
Created: 3/3/2025
Status: Merged
Merged: 3/3/2025
Merged by: @Bekacru

Base: mainHead: patch-1


📝 Commits (1)

  • 57bf02f use global name for discord

📊 Changes

1 file changed (+1 additions, -1 deletions)

View changed files

📝 packages/better-auth/src/social-providers/discord.ts (+1 -1)

📄 Description

display_name seems to not be present anymore, keeping it in the type just incase im just not seeing it on my own personal discord account

this matches the way next auth handles discord names here


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/better-auth/better-auth/pull/1656 **Author:** [@juliesaia](https://github.com/juliesaia) **Created:** 3/3/2025 **Status:** ✅ Merged **Merged:** 3/3/2025 **Merged by:** [@Bekacru](https://github.com/Bekacru) **Base:** `main` ← **Head:** `patch-1` --- ### 📝 Commits (1) - [`57bf02f`](https://github.com/better-auth/better-auth/commit/57bf02f67034865b4ae3c481d2c61ef2a1e6ebd4) use global name for discord ### 📊 Changes **1 file changed** (+1 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `packages/better-auth/src/social-providers/discord.ts` (+1 -1) </details> ### 📄 Description `display_name` seems to not be present anymore, keeping it in the type just incase im just not seeing it on my own personal discord account this matches the way next auth handles discord names [here](https://github.com/nextauthjs/next-auth/blob/main/packages/core/src/providers/discord.ts) --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
GiteaMirror added the pull-request label 2026-03-13 11:22:51 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/better-auth#3926