Skip to content

fix(projects): confirm before discarding unsaved project edits - #148

Merged
JoshuaKGoldberg merged 1 commit into
masterfrom
confirm-discarding-project-form
Aug 18, 2026
Merged

fix(projects): confirm before discarding unsaved project edits#148
JoshuaKGoldberg merged 1 commit into
masterfrom
confirm-discarding-project-form

Conversation

@JoshuaKGoldberg

@JoshuaKGoldberg JoshuaKGoldberg commented Aug 18, 2026

Copy link
Copy Markdown
Member

Never mind on the project form called onCancel with no dirty check, so one stray click silently threw away everything typed in. (slack thread)

The form now:

  • Derives its starting values through a single initialValues(project, claim) helper used both for the useState initializers and for a dirty comparison
  • Has Never mind ask with a modal before discarding when anything differs

🤖 Generated with Claude Code

`Never mind` called `onCancel` immediately, so a stray click threw away
everything typed into the project form. It now asks for confirmation
whenever the form differs from the values it started with.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@JoshuaKGoldberg
JoshuaKGoldberg marked this pull request as ready for review August 18, 2026 16:10
@HazAT
HazAT self-requested a review August 18, 2026 18:55
@JoshuaKGoldberg
JoshuaKGoldberg merged commit 1e67c59 into master Aug 18, 2026
13 checks passed
@JoshuaKGoldberg
JoshuaKGoldberg deleted the confirm-discarding-project-form branch August 18, 2026 21:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants