[PR #482] [CLOSED] Draft: Use GORM to maintain multi-database layer #609

New issue

Closed

opened 2026-02-25 23:34:53 +03:00 by kerem · 0 comments

kerem commented

2026-02-25 23:34:53 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/go-shiori/shiori/pull/482
Author: @fmartingr
Created: 9/30/2022
Status: ❌ Closed

Base: master ← Head: draft/gorm

📝 Commits (3)

489f0ef gorm models
e9b61fc gorm adapter wip
cf1ceaf modules: add gorm

📊 Changes

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

View changed files

📝 go.mod (+5 -0)
📝 go.sum (+10 -0)
📝 internal/cmd/root.go (+6 -0)
➕ internal/database/sqlite_gorm.go (+117 -0)
📝 internal/model/model.go (+39 -22)

📄 Description

Experiment to incorporate gorm into Shiori to take care of all the database logic, instead of having to maintain three different implementations, migrations and logic.

_{🔄 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-shiori/shiori/pull/482 **Author:** [@fmartingr](https://github.com/fmartingr) **Created:** 9/30/2022 **Status:** ❌ Closed **Base:** `master` ← **Head:** `draft/gorm` --- ### 📝 Commits (3) - [`489f0ef`](https://github.com/go-shiori/shiori/commit/489f0ef5d39fbcc46e61117f8038e1f759582db6) gorm models - [`e9b61fc`](https://github.com/go-shiori/shiori/commit/e9b61fcaf982ae197d7821c5176ba2bde1186567) gorm adapter wip - [`cf1ceaf`](https://github.com/go-shiori/shiori/commit/cf1ceaf98c703c80f2bbf5150764537cda57aa1d) modules: add gorm ### 📊 Changes **5 files changed** (+177 additions, -22 deletions) <details> <summary>View changed files</summary> 📝 `go.mod` (+5 -0) 📝 `go.sum` (+10 -0) 📝 `internal/cmd/root.go` (+6 -0) ➕ `internal/database/sqlite_gorm.go` (+117 -0) 📝 `internal/model/model.go` (+39 -22) </details> ### 📄 Description Experiment to incorporate [gorm](https://gorm.io/) into Shiori to take care of all the database logic, instead of having to maintain three different implementations, migrations and logic. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>