[PR #1656] [MERGED] Add primary key and index to external login user table #15977

Closed
opened 2025-11-02 11:59:25 -06:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/go-gitea/gitea/pull/1656
Author: @lafriks
Created: 5/1/2017
Status: Merged
Merged: 5/2/2017
Merged by: @appleboy

Base: masterHead: issue/1655


📝 Commits (1)

  • 50406f6 Add primary key and index to external login user table

📊 Changes

3 files changed (+43 additions, -3 deletions)

View changed files

📝 models/external_login_user.go (+3 -3)
📝 models/migrations/migrations.go (+2 -0)
models/migrations/v30.go (+38 -0)

📄 Description

Fixes #1655

I don't know if that's the best way for migration, it would be better to add primary key as SQL commands not recreating table, let me know what you think.


🔄 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/go-gitea/gitea/pull/1656 **Author:** [@lafriks](https://github.com/lafriks) **Created:** 5/1/2017 **Status:** ✅ Merged **Merged:** 5/2/2017 **Merged by:** [@appleboy](https://github.com/appleboy) **Base:** `master` ← **Head:** `issue/1655` --- ### 📝 Commits (1) - [`50406f6`](https://github.com/go-gitea/gitea/commit/50406f61f49973bc74f9d9d28b36e9e968010ee8) Add primary key and index to external login user table ### 📊 Changes **3 files changed** (+43 additions, -3 deletions) <details> <summary>View changed files</summary> 📝 `models/external_login_user.go` (+3 -3) 📝 `models/migrations/migrations.go` (+2 -0) ➕ `models/migrations/v30.go` (+38 -0) </details> ### 📄 Description Fixes #1655 I don't know if that's the best way for migration, it would be better to add primary key as SQL commands not recreating table, let me know what you think. --- <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 2025-11-02 11:59:25 -06:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/gitea#15977