feat(ui): always show Remove from DB button for DB-sourced settings with confirmation dialog

Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com>
This commit is contained in:
copilot-swe-agent[bot]
2026-02-23 12:26:45 +00:00
parent 862ba99802
commit 1cf6adbc7f
+6 -2
View File
@@ -201,14 +201,14 @@
</button> </button>
<button <button
type="button" type="button"
x-show="isDbOverride['{{ setting.key }}'] && formData['{{ setting.key }}'] === originalData['{{ setting.key }}']" x-show="isDbOverride['{{ setting.key }}']"
x-transition x-transition
@click="revertSetting('{{ setting.key }}')" @click="revertSetting('{{ setting.key }}')"
:disabled="revertingKey === '{{ setting.key }}'" :disabled="revertingKey === '{{ setting.key }}'"
class="px-3 py-1 text-sm bg-orange-500 text-white rounded-md hover:bg-orange-600 focus:outline-none focus:ring-2 focus:ring-offset-2 focus:ring-orange-400 disabled:opacity-50 disabled:cursor-not-allowed whitespace-nowrap" class="px-3 py-1 text-sm bg-orange-500 text-white rounded-md hover:bg-orange-600 focus:outline-none focus:ring-2 focus:ring-offset-2 focus:ring-orange-400 disabled:opacity-50 disabled:cursor-not-allowed whitespace-nowrap"
title="Remove DB override and revert to environment variable or default" title="Remove DB override and revert to environment variable or default"
> >
<span x-show="revertingKey !== '{{ setting.key }}'"><i class="fas fa-undo mr-1"></i>Back to ENV</span> <span x-show="revertingKey !== '{{ setting.key }}'"><i class="fas fa-trash-alt mr-1"></i>Remove from DB</span>
<span x-show="revertingKey === '{{ setting.key }}'">Reverting…</span> <span x-show="revertingKey === '{{ setting.key }}'">Reverting…</span>
</button> </button>
</div> </div>
@@ -330,6 +330,10 @@ function settingsApp() {
}, },
async revertSetting(key) { async revertSetting(key) {
if (!confirm(`Remove '${key}' from the database?\n\nThe setting will revert to its environment variable or default value.`)) {
return;
}
this.revertingKey = key; this.revertingKey = key;
this.hideAlert(); this.hideAlert();