-
Notifications
You must be signed in to change notification settings - Fork 8
Setup Wizard: Form Importers #1005
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Setup Wizard: Form Importers #1005
Conversation
WordPress Playground🚀 Your PR has been built and is ready for testing in WordPress Playground! |
…etup-wizard-form-importers
Campaign Monitor uses e.g. `cm_123456`
…o importer-campaign-monitor
WordPress Playground🚀 Your PR has been built and is ready for testing in WordPress Playground! |
ciccio-kit
left a comment
There was a problem hiding this 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'] ) ); |
There was a problem hiding this comment.
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
Importer: Campaign Monitor
WordPress Playground🚀 Your PR has been built and is ready for testing in WordPress Playground! |
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:
Testing
testSetupWizardFormImporterScreen: Test that the Form Importer step is available when a supported third party form plugin has form(s) on the WordPress siteChecklist