Skip to content

Subscribe Form and Advertisement blocks bug fixes#146

Merged
zackaira merged 5 commits into
mainfrom
fix/subscribe-form-and-advertisement-blocks
Jul 7, 2026
Merged

Subscribe Form and Advertisement blocks bug fixes#146
zackaira merged 5 commits into
mainfrom
fix/subscribe-form-and-advertisement-blocks

Conversation

The beehiiv subscribe form preview did not render correctly with different
form styles in the block editor. Replace the SandBox preview with a
placeholder explaining the form only renders on the front-end, and remove
the now-unused preview markup and styles.
An empty Advertisement block was silently dropped from the payload, so
the newsletter sent without it as long as another supported block was
present. Reject the send instead and tell the user to select an ad or
remove the block.

- Refuse the send server-side with a beehiiv_advertisement_no_ad error
  and editor-friendly copy
- Show a proactive sidebar notice while "Send to newsletter" is on,
  suppressed once the server-side error is displayed to avoid duplicates
A selected advertisement stayed in the block dropdown as a normal option
after it was unclaimed/removed in beehiiv, with no indication it was gone,
letting a newsletter send with a reference to an ad that no longer exists.

- Label the saved selection '— no longer available' in the dropdown when a
  successful load omits it, guarding on a load-success flag so a failed or
  empty error response never false-flags an ad as stale
- Add a warning notice with a one-click 'Clear selection' action
- Refuse the send server-side with a friendly error when an Advertisement
  block references an ad beehiiv no longer offers, enforced only on a
  confirmed API response so transient failures never block a valid send
@chandrapatel
chandrapatel requested a review from zackaira July 6, 2026 16:42
@zackaira
zackaira merged commit 95edc3d into main Jul 7, 2026
8 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.

2 participants