Skip to content

Form Builder: Set inactive subscriber state when a Kit Form is used#1084

Merged
n7studios merged 12 commits intomainfrom
form-builder-form-subscriber-state
May 5, 2026
Merged

Form Builder: Set inactive subscriber state when a Kit Form is used#1084
n7studios merged 12 commits intomainfrom
form-builder-form-subscriber-state

Conversation

@n7studios
Copy link
Copy Markdown
Contributor

@n7studios n7studios commented Apr 30, 2026

Summary

Fixes this reported issue by creating a subscriber with state of inactive when the Form Builder block is set to subscribe the email address to a Kit Form.

This allows the form's single / double opt in to be honored, which creators expect, and matches how this works for Contact Form 7, Forminator and WishList Member.

Testing

  • testFormBuilderBlockWithSingleOptinFormEnabled: Test the Form Builder block works when added and that the subscriber's state is active when a single optin Form is specified.
  • testFormBuilderBlockWithDoubleOptinFormEnabled: Test the Form Builder block works when added and that the subscriber's state is inactive when a double optin Form is specified.

Checklist

@n7studios n7studios self-assigned this Apr 30, 2026
@n7studios n7studios added the bug label Apr 30, 2026
@github-actions
Copy link
Copy Markdown

WordPress Playground

🚀 Your PR has been built and is ready for testing in WordPress Playground!

Click here to test your changes in WordPress Playground

@n7studios n7studios requested review from a team, ciccio-kit and noelherrick and removed request for a team April 30, 2026 05:56
@n7studios n7studios marked this pull request as ready for review April 30, 2026 05:56
Copy link
Copy Markdown
Contributor

@noelherrick noelherrick left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you write a test that makes sure that the subscriber is active on a single-opt-in form? I can see that the forms endpoint should handle this, but since you're using the subscriber endpoint, I just want to confirm it will follow the form activation rules

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 1, 2026

WordPress Playground

🚀 Your PR has been built and is ready for testing in WordPress Playground!

Click here to test your changes in WordPress Playground

@n7studios n7studios added this to the 3.3.2 milestone May 4, 2026
@n7studios n7studios requested a review from noelherrick May 4, 2026 02:55
n7studios added 2 commits May 5, 2026 10:09
Tests: Integrations: Set `inactive` subscriber state when a Kit Form is used
@n7studios n7studios merged commit f8ae5d8 into main May 5, 2026
@n7studios n7studios deleted the form-builder-form-subscriber-state branch May 5, 2026 06:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants