[PR #220] [MERGED] User::save() should return QueryResult instead of bool #2674

New issue

Closed

opened 2026-03-03 08:36:27 +03:00 by kerem · 0 comments

kerem commented

2026-03-03 08:36:27 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/dani-garcia/vaultwarden/pull/220
Author: @janost
Created: 10/14/2018
Status: ✅ Merged
Merged: 10/14/2018
Merged by: @dani-garcia

Base: master ← Head: refactor-user-save

📝 Commits (1)

e985221 User::save() should return QueryResult instead of bool

📊 Changes

5 files changed (+47 additions, -37 deletions)

View changed files

📝 src/api/core/accounts.rs (+30 -21)
📝 src/api/core/mod.rs (+4 -2)
📝 src/api/core/organizations.rs (+3 -3)
📝 src/api/core/two_factor.rs (+7 -4)
📝 src/db/models/user.rs (+3 -7)

📄 Description

Related to #6.

_{🔄 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/dani-garcia/vaultwarden/pull/220 **Author:** [@janost](https://github.com/janost) **Created:** 10/14/2018 **Status:** ✅ Merged **Merged:** 10/14/2018 **Merged by:** [@dani-garcia](https://github.com/dani-garcia) **Base:** `master` ← **Head:** `refactor-user-save` --- ### 📝 Commits (1) - [`e985221`](https://github.com/dani-garcia/vaultwarden/commit/e985221b50d42a9c43d3f958a1de4ff9f69eb351) User::save() should return QueryResult instead of bool ### 📊 Changes **5 files changed** (+47 additions, -37 deletions) <details> <summary>View changed files</summary> 📝 `src/api/core/accounts.rs` (+30 -21) 📝 `src/api/core/mod.rs` (+4 -2) 📝 `src/api/core/organizations.rs` (+3 -3) 📝 `src/api/core/two_factor.rs` (+7 -4) 📝 `src/db/models/user.rs` (+3 -7) </details> ### 📄 Description Related to #6. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>