Skip to content

fix(web): skip untouched empty compose drafts - #187

Merged
Tutitoos merged 1 commit into
mainfrom
bugfix/186-empty-compose-draft
Sep 11, 2026
Merged

Tutitoos merged 1 commit into
mainfrom
bugfix/186-empty-compose-draft

Conversation

@Tutitoos

Copy link
Copy Markdown
Owner

Summary

  • avoid autosaving or checkpointing a brand-new composer until it contains a recipient, subject, body text, or attachment
  • close an untouched composer without creating a server-side draft
  • preserve recovered draft, reply/forward, autosave, checkpoint, and send behavior once content exists
  • cover the empty compose path in unit and six-viewport browser tests

Validation

  • bunx biome check apps/web/src/composer.tsx apps/web/src/composer.test.ts apps/web/tests/inbox.spec.ts
  • bun run --cwd apps/web typecheck
  • bun run --cwd apps/web test (13 files, 50 tests)
  • bun run --cwd apps/web build
  • bun run --cwd apps/web test:e2e -- --project=tablet-portrait --project=mobile-large --project=mobile tests/inbox.spec.ts
  • bun run --cwd apps/web test:e2e -- --project=desktop-large --project=desktop --project=tablet-landscape tests/inbox.spec.ts

Closes #186

@Tutitoos Tutitoos added this to the Phase 8 — Release 1.0 milestone Sep 11, 2026
@Tutitoos Tutitoos added javascript Pull requests that update javascript code type:bug Normal defect in the regular delivery flow area:web Web application labels Sep 11, 2026
@Tutitoos
Tutitoos merged commit 07a2d03 into main Sep 11, 2026
18 checks passed
@Tutitoos
Tutitoos deleted the bugfix/186-empty-compose-draft branch September 11, 2026 03:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:web Web application javascript Pull requests that update javascript code type:bug Normal defect in the regular delivery flow

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix(web): do not persist untouched empty compose drafts

1 participant