Skip to content

Commit ec63788

Browse files
committed
refactor: simplify save function in SiteAdd.vue to use async/await for better readability and error handling
1 parent bb18ac8 commit ec63788

File tree

1 file changed

+12
-13
lines changed

1 file changed

+12
-13
lines changed

app/src/views/site/site_add/SiteAdd.vue

Lines changed: 12 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -26,20 +26,19 @@ function init() {
2626
}
2727
2828
async function save() {
29-
return ngx.build_config(ngxConfig.value).then(r => {
30-
site.updateItem(ngxConfig.value.name, { name: ngxConfig.value.name, content: r.content, overwrite: false }).then(() => {
31-
message.success($gettext('Saved successfully'))
32-
33-
site.enable(ngxConfig.value.name).then(() => {
34-
message.success($gettext('Enabled successfully'))
35-
window.scroll({ top: 0, left: 0, behavior: 'smooth' })
36-
}).catch(e => {
37-
message.error(e.message ?? $gettext('Enable failed'), 5)
38-
})
39-
}).catch(e => {
40-
message.error($gettext('Save error %{msg}', { msg: $gettext(e.message) ?? '' }), 5)
41-
})
29+
const r = await ngx.build_config(ngxConfig.value)
30+
await site.updateItem(ngxConfig.value.name, {
31+
name: ngxConfig.value.name,
32+
content: r.content,
33+
overwrite: currentStep.value === 1, // only overwrite when step 2 is completed
4234
})
35+
36+
message.success($gettext('Saved successfully'))
37+
38+
await site.enable(ngxConfig.value.name)
39+
message.success($gettext('Enabled successfully'))
40+
41+
window.scroll({ top: 0, left: 0, behavior: 'smooth' })
4342
}
4443
4544
const router = useRouter()

0 commit comments

Comments
 (0)