Since the change to camelCase variables the error messages in the Vaultwarden Admin were not shown correctly anymore.
This PR fixes this by changing the case of the json key's. Also updated the save and delete of the config to provide a more descriptive error instead of only Io or which ever other error might occure.
🔄 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/4869
**Author:** [@BlackDex](https://github.com/BlackDex)
**Created:** 8/18/2024
**Status:** ✅ Merged
**Merged:** 8/18/2024
**Merged by:** [@dani-garcia](https://github.com/dani-garcia)
**Base:** `main` ← **Head:** `fix-admin-error-messages`
---
### 📝 Commits (1)
- [`e190648`](https://github.com/dani-garcia/vaultwarden/commit/e190648f029a0add07732e3420ebff93df40edfb) Fix Vaultwarden Admin page error messages
### 📊 Changes
**2 files changed** (+10 additions, -4 deletions)
<details>
<summary>View changed files</summary>
📝 `src/api/admin.rs` (+8 -2)
📝 `src/static/scripts/admin.js` (+2 -2)
</details>
### 📄 Description
Since the change to camelCase variables the error messages in the Vaultwarden Admin were not shown correctly anymore.
This PR fixes this by changing the case of the json key's. Also updated the save and delete of the config to provide a more descriptive error instead of only `Io` or which ever other error might occure.
Fixes #4834
---
<sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
📋 Pull Request Information
Original PR: https://github.com/dani-garcia/vaultwarden/pull/4869
Author: @BlackDex
Created: 8/18/2024
Status: ✅ Merged
Merged: 8/18/2024
Merged by: @dani-garcia
Base:
main← Head:fix-admin-error-messages📝 Commits (1)
e190648Fix Vaultwarden Admin page error messages📊 Changes
2 files changed (+10 additions, -4 deletions)
View changed files
📝
src/api/admin.rs(+8 -2)📝
src/static/scripts/admin.js(+2 -2)📄 Description
Since the change to camelCase variables the error messages in the Vaultwarden Admin were not shown correctly anymore.
This PR fixes this by changing the case of the json key's. Also updated the save and delete of the config to provide a more descriptive error instead of only
Ioor which ever other error might occure.Fixes #4834
🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.