Skip to content

Conversation

@n7studios
Copy link
Contributor

@n7studios n7studios commented Feb 2, 2026

Summary

Displays a screen during the onboarding/setup wizard, prompting users to automatically replace third party forms with Kit Forms across their WordPress web site, if third party forms are detected on the site based on the current registered importers the Kit Plugin has:

Screenshot 2026-02-02 at 18 02 04

Testing

  • testSetupWizardFormImporterScreen: Test that the Form Importer step is available when a supported third party form plugin has form(s) on the WordPress site

Checklist

@n7studios n7studios self-assigned this Feb 2, 2026
@github-actions
Copy link

github-actions bot commented Feb 3, 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 requested review from a team, ciccio-kit and noelherrick and removed request for a team February 3, 2026 03:52
@n7studios n7studios marked this pull request as ready for review February 3, 2026 03:52
@n7studios n7studios marked this pull request as draft February 4, 2026 09:38
@github-actions
Copy link

github-actions bot commented Feb 5, 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 requested review from a team, ciccio-kit and noelherrick and removed request for a team February 5, 2026 07:35
@n7studios n7studios marked this pull request as ready for review February 5, 2026 07:35
Copy link

@ciccio-kit ciccio-kit left a comment

Choose a reason for hiding this comment

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

Found only one potential issue, but pre-approving. When fixed (or if not relevant), feel free to go ahead and merge ✅

if ( array_key_exists( 'error', $request ) || array_key_exists( 'error_description', $request ) ) {
// Decrement the step.
$this->step = 'start';
$this->error = sanitize_text_field( wp_unslash( $request['error_description'] ) );

Choose a reason for hiding this comment

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

issue: based on the OR in this if there is a risk this line is reached even if there is no error_description in $request. If this is a possible scenario it should be handled safely

@n7studios n7studios merged commit dcb0b32 into setup-wizard-named-steps Feb 7, 2026
83 of 86 checks passed
@github-actions
Copy link

github-actions bot commented Feb 7, 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 deleted the setup-wizard-form-importers branch February 10, 2026 06:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants