Skip to content

fix(forms): emit content event for public submissions - #471

Merged
DavidBabinec merged 1 commit into
mainfrom
fix/public-form-content-events
Aug 31, 2026
Merged

fix(forms): emit content event for public submissions#471
DavidBabinec merged 1 commit into
mainfrom
fix/public-form-content-events

Conversation

@DavidBabinec

Copy link
Copy Markdown
Contributor

Fixes #372

What changed

  • Emit content.entry.created after a successful public form row is persisted.
  • Use the existing centralized content-event helper and { kind: 'system' } actor.
  • Extend the real challenged-submission test to verify the table slug, row ID, and actor payload.
  • Document public submissions in the forms, content storage, and plugin event contracts.

Why

The public form handler returned immediately after creating its row, bypassing the event channel used by every other content creation path. Notification and automation plugins could not observe successful submissions. Rejected-submission observability remains separate from this fix.

Verification

  • Before: targeted test failed, 12 pass / 1 fail
  • Fixed: targeted test passed, 13 pass / 0 fail
  • Control without handler fix: failed, 12 pass / 1 fail
  • bun run build and bun run lint (exit 0)
  • bun test (exit 0, 6,725 pass / 0 fail)

@DavidBabinec
DavidBabinec marked this pull request as ready for review August 31, 2026 21:09
@DavidBabinec
DavidBabinec merged commit fd7cb2b into main Aug 31, 2026
9 checks passed
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.

[Bug]: Public form submissions bypass the content.entry.* plugin event channel

1 participant