[PR #474] [MERGED] fix: remove email field from member #3315

Closed
opened 2026-03-13 10:48:43 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/better-auth/better-auth/pull/474
Author: @Bekacru
Created: 11/9/2024
Status: Merged
Merged: 11/10/2024
Merged by: @Bekacru

Base: mainHead: refactor/organization


📝 Commits (4)

  • f5c8f57 # This is a combination of 3 commits.
  • 7313bf4 chore: lock file
  • c29eb51 fix(organization): remove email field from member
  • 4523ff5 Merge branch 'main' into refactor/organization

📊 Changes

5 files changed (+15 additions, -22 deletions)

View changed files

📝 packages/better-auth/src/plugins/organization/adapter.ts (+12 -13)
📝 packages/better-auth/src/plugins/organization/organization.test.ts (+1 -2)
📝 packages/better-auth/src/plugins/organization/organization.ts (+0 -4)
📝 packages/better-auth/src/plugins/organization/routes/crud-invites.ts (+2 -2)
📝 packages/better-auth/src/plugins/organization/schema.ts (+0 -1)

📄 Description

The email field is unnecessary in the member table since we can always reference it from the user table. Including it only complicates email updates.


🔄 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/474 **Author:** [@Bekacru](https://github.com/Bekacru) **Created:** 11/9/2024 **Status:** ✅ Merged **Merged:** 11/10/2024 **Merged by:** [@Bekacru](https://github.com/Bekacru) **Base:** `main` ← **Head:** `refactor/organization` --- ### 📝 Commits (4) - [`f5c8f57`](https://github.com/better-auth/better-auth/commit/f5c8f57dfd569470b499aca3fea5de89694a1a78) # This is a combination of 3 commits. - [`7313bf4`](https://github.com/better-auth/better-auth/commit/7313bf4b86010d75824e4fa8a48f74068777d03a) chore: lock file - [`c29eb51`](https://github.com/better-auth/better-auth/commit/c29eb518be9eba52ba87d6d6218e5e7f4ad6b69a) fix(organization): remove email field from member - [`4523ff5`](https://github.com/better-auth/better-auth/commit/4523ff52911a827ffbb3dd75fc09e1fdbc971abe) Merge branch 'main' into refactor/organization ### 📊 Changes **5 files changed** (+15 additions, -22 deletions) <details> <summary>View changed files</summary> 📝 `packages/better-auth/src/plugins/organization/adapter.ts` (+12 -13) 📝 `packages/better-auth/src/plugins/organization/organization.test.ts` (+1 -2) 📝 `packages/better-auth/src/plugins/organization/organization.ts` (+0 -4) 📝 `packages/better-auth/src/plugins/organization/routes/crud-invites.ts` (+2 -2) 📝 `packages/better-auth/src/plugins/organization/schema.ts` (+0 -1) </details> ### 📄 Description The `email` field is unnecessary in the member table since we can always reference it from the `user` table. Including it only complicates email updates. --- <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 10:48:43 -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#3315