[PR #382] [MERGED] server,app: migrate to sea-orm #710

New issue

Closed

opened 2026-02-27 09:09:27 +03:00 by kerem · 0 comments

kerem commented

2026-02-27 09:09:27 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/lldap/lldap/pull/382
Author: @nitnelave
Created: 11/25/2022
Status: ✅ Merged
Merged: 11/25/2022
Merged by: @nitnelave

Base: main ← Head: orm

📝 Commits (2)

deb0e9e server,app: migrate to sea-orm
dd4e11e server: move domain types to a separate file

📊 Changes

45 files changed (+2375 additions, -1642 deletions)

View changed files

📝 Cargo.lock (+436 -148)
📝 Cargo.toml (+4 -0)
📝 app/Cargo.toml (+1 -0)
📝 app/src/components/create_user.rs (+1 -4)
📝 app/src/components/group_details.rs (+1 -1)
📝 app/src/components/group_table.rs (+1 -1)
📝 app/src/components/user_details_form.rs (+7 -7)
📝 app/src/components/user_table.rs (+1 -1)
📝 app/src/infra/cookies.rs (+5 -1)
📝 schema.graphql (+1 -1)
📝 server/Cargo.toml (+7 -19)
📝 server/src/domain/error.rs (+3 -1)
📝 server/src/domain/handler.rs (+8 -217)
📝 server/src/domain/ldap/group.rs (+2 -2)
📝 server/src/domain/ldap/user.rs (+6 -6)
📝 server/src/domain/ldap/utils.rs (+2 -4)
📝 server/src/domain/mod.rs (+2 -0)
➕ server/src/domain/model/groups.rs (+53 -0)
➕ server/src/domain/model/jwt_refresh_storage.rs (+35 -0)
➕ server/src/domain/model/jwt_storage.rs (+36 -0)

...and 25 more files

📄 Description

This PR migrates the project to use sea-orm instead of the lower-level sea-query.

The first-order effect is that it fixes #87 by being database-agnostic.
The second-order effect is that it makes complex queries (especially multi-table complex joins) easier to write. This will be especially useful for #67.

_{🔄 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/lldap/lldap/pull/382 **Author:** [@nitnelave](https://github.com/nitnelave) **Created:** 11/25/2022 **Status:** ✅ Merged **Merged:** 11/25/2022 **Merged by:** [@nitnelave](https://github.com/nitnelave) **Base:** `main` ← **Head:** `orm` --- ### 📝 Commits (2) - [`deb0e9e`](https://github.com/lldap/lldap/commit/deb0e9ec4bcba8fcb88210855d2bfde61fda041a) server,app: migrate to sea-orm - [`dd4e11e`](https://github.com/lldap/lldap/commit/dd4e11e72f590584c9f98d982072932a006612de) server: move domain types to a separate file ### 📊 Changes **45 files changed** (+2375 additions, -1642 deletions) <details> <summary>View changed files</summary> 📝 `Cargo.lock` (+436 -148) 📝 `Cargo.toml` (+4 -0) 📝 `app/Cargo.toml` (+1 -0) 📝 `app/src/components/create_user.rs` (+1 -4) 📝 `app/src/components/group_details.rs` (+1 -1) 📝 `app/src/components/group_table.rs` (+1 -1) 📝 `app/src/components/user_details_form.rs` (+7 -7) 📝 `app/src/components/user_table.rs` (+1 -1) 📝 `app/src/infra/cookies.rs` (+5 -1) 📝 `schema.graphql` (+1 -1) 📝 `server/Cargo.toml` (+7 -19) 📝 `server/src/domain/error.rs` (+3 -1) 📝 `server/src/domain/handler.rs` (+8 -217) 📝 `server/src/domain/ldap/group.rs` (+2 -2) 📝 `server/src/domain/ldap/user.rs` (+6 -6) 📝 `server/src/domain/ldap/utils.rs` (+2 -4) 📝 `server/src/domain/mod.rs` (+2 -0) ➕ `server/src/domain/model/groups.rs` (+53 -0) ➕ `server/src/domain/model/jwt_refresh_storage.rs` (+35 -0) ➕ `server/src/domain/model/jwt_storage.rs` (+36 -0) _...and 25 more files_ </details> ### 📄 Description This PR migrates the project to use sea-orm instead of the lower-level sea-query. The first-order effect is that it fixes #87 by being database-agnostic. The second-order effect is that it makes complex queries (especially multi-table complex joins) easier to write. This will be especially useful for #67. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>