spikerj/spikersoft-issues
1
0 Fork 0
Code Issues 23 Pull Requests Actions Packages Projects Releases Wiki Activity

[UX][Blog][Angular] Create flow: setTimeout(1000) navigation + no unsaved-changes guard (epic #705) #712

New Issue
Closed
opened 2026-07-19 01:15:37 +00:00 by spikerj · 1 comment
spikerj commented 2026-07-19 01:15:37 +00:00
Owner
Copy link
Copy source

Epic #705. blog-create.ts: after publish/save-draft the component navigates back to /blog via a bare setTimeout(..., 1000) — fragile timing (double-submits possible in the window, navigation races the snackbar) — and the form has no CanDeactivate/unsaved-changes guard, so a stray back-press discards a fully-written post silently.

Shape: navigate in the subscribe next (keep the snackbar; no timer), disable the submit while in flight, add a canDeactivate guard prompting when title/body/media are dirty (there are existing guard patterns in the app to mirror).

Epic #705. `blog-create.ts`: after publish/save-draft the component navigates back to `/blog` via a bare `setTimeout(..., 1000)` — fragile timing (double-submits possible in the window, navigation races the snackbar) — and the form has **no CanDeactivate/unsaved-changes guard**, so a stray back-press discards a fully-written post silently. **Shape:** navigate in the subscribe `next` (keep the snackbar; no timer), disable the submit while in flight, add a `canDeactivate` guard prompting when title/body/media are dirty (there are existing guard patterns in the app to mirror).
spikerj commented 2026-08-07 13:46:03 +00:00
Author
Owner
Copy link
Copy source

Migrated to spikerj/spikersoft-angular#631 as part of the umbrella-tracker breakup.

Verified 2026-08-07 against spikersoft-angular@8e5a4048, and live against the deployed learn.spikersoft.com bundle (main-KZKZUOSX.js).
Status: not started — All of it: navigate in the subscribe next, disable submit while in flight, add the canDeactivate guard.

Closing here. Work now lives in the repo that holds the fix, so fixes #631 in a PR will auto-close it on merge. The umbrella tracker keeps cross-repo epics only (parent epic #705 is untouched).

— Opus 5 Agent

Migrated to **spikerj/spikersoft-angular#631** as part of the umbrella-tracker breakup. Verified 2026-08-07 against `spikersoft-angular@8e5a4048`, and live against the deployed `learn.spikersoft.com` bundle (`main-KZKZUOSX.js`). Status: not started — All of it: navigate in the subscribe `next`, disable submit while in flight, add the `canDeactivate` guard. Closing here. Work now lives in the repo that holds the fix, so `fixes #631` in a PR will auto-close it on merge. The umbrella tracker keeps cross-repo epics only (parent epic #705 is untouched). — Opus 5 Agent
Sign in to join this conversation.
No Branch/Tag Specified
Labels
Clear labels
agentic
Actionable surfaced by the agentic architecture & security review
 bug
Something is not working
 duplicate
This issue or pull request already exists
 enhancement
New feature
 epic
Cross-repo epic — children live in per-repo trackers
 help wanted
Need some help
 invalid
Something is wrong
 question
More information is needed
 sonarqube
Code quality issue surfaced by SonarQube (bug / code smell)
 wontfix
This won't be fixed
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
spikerj (Joseph Spiker)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: spikerj/spikersoft-issues#712