lfg2025/deepstock
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: don't require report_id/token on submit success

Browse Source 
The deployed storage-free worker returns { ok: true } without
report_id/report_token, but the submit handler threw 'Your plan could
not be saved' whenever they were absent — showing an error even though
the email sent. Treat the worker's ok response as success and only
capture the tokens when the D1 worker provides them (restore link).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This commit is contained in:
Dorian
2026-06-16 15:09:45 +01:00
parent 0d3efe4139
commit b845957314
1 changed files with 12 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

17
src/App.vue
View File

@@ -1779,13 +1779,20 @@ function submitCapture(e) {
return res.json() return res.json()
}) })
.then(data => { .then(data => {
if (!data || !data.report_id || !data.report_token) { // The HTTP res.ok check above already confirmed the worker accepted
// the submission (email sent). report_id/report_token are only
// returned by the D1-backed worker; capture them when present so the
// email restore link works, but don't require them — the storage-free
// worker sends the plan without persisting anything.
if (data && data.ok === false) {
throw new Error(currentLang === 'de' throw new Error(currentLang === 'de'
? 'Dein Plan konnte gerade nicht gespeichert werden. Bitte versuche es erneut.' ? 'Dein Plan konnte gerade nicht per E-Mail gesendet werden. Bitte versuche es erneut.'
: 'Your plan could not be saved just now. Please try again.') : 'Your plan could not be emailed just now. Please try again.')
}
if (data && data.report_id && data.report_token) {
answers._report_id = String(data.report_id)
answers._report_token = String(data.report_token)
} }
answers._report_id = String(data.report_id)
answers._report_token = String(data.report_token)
saveState() saveState()
showSuccess() showSuccess()
}) })