Evf 2106 introducing fix email delivery wizard or setup#1553
Draft
som3669 wants to merge 24 commits into
Draft
Conversation
* add option to download and copy the logs * fix: log button responsive with issue * update:review comment
* add: integrations and payments tab * sidebar design update * update: integration design * update: opacity and icon * remove unwanted class * remove unwanted class * fix text domain * fix class issue * fix class exist * fix duplicate class issue
…lexibility. (#1542) * option to send form static reporting at a time * fix send test report email address update issue * remove: nps * fix two time mail send while sending report mail * update: review report * update: review report
* Fix - Coupon limit validation * fix: coupons not applied issue * pass: coupon data for coupon calculation on backend * pass: all coupon data for the coupon validation * merge pre-develop into EVF-2293-enhance/coupons * Dev - Payment Summary Field. (#1545) * added: payment sumary description * fix: show payment history field on pro version * fix: saving the form due to the payment summary * added: show hide button for the payment history * fix: class not found issue * changelog updated * changelog updated
* Initial Commit * UI updates for field adding popover * Placement updates for fields popover * Popover added for row add button too * Ui updates for popover trigger button * Popover arrow UI added * Removed non-important comments * Layout options such as one-column container, two-column container, etc. * Enable grouping feature for fields. * fix : Field still hoverable after Selected * fix : Field popover placement issue fixed * fix: popup clipping issue * update: update selected count while duplicate * update:on group delete redirect to add field --------- Co-authored-by: som3669 <[email protected]>
…very tracking - Add is_smart_smtp_installed() and is_smart_smtp_active() helpers to EVF_Site_Assistant - Extend GET /site-assistant and skip-setup responses with last_form_email_status, is_smart_smtp_installed, is_smart_smtp_active - Change send_test_email endpoint to always return HTTP 200 JSON with email_sent bool instead of WP_Error 500 on failure - Write everest_forms_last_form_email_status WP option in EVF_Emails::send() on every attempt - Add PHPUnit tests for all new API fields and email status tracking Co-Authored-By: Claude Sonnet 4.6 <[email protected]>
…art SMTP install Co-Authored-By: Claude Sonnet 4.6 <[email protected]>
…t Email wizard step
- Add emailStatus state ('idle'|'sent'|'failed') driven by sendTestEmailMutation response
- Initialize emailStatus from last_form_email_status server state on mount
- Show inline red Alert on failure: 'Test Email Failed'
- Show inline green Alert on success: 'Test Email Sent Successfully'
- Show Smart SMTP install card when status is set and Smart SMTP is not active
- 'Fix this with SmartSMTP' on failure, 'Want more reliable delivery?' on success
- 'Install Plugin'/'Activate Plugin' button triggers inline AJAX install
- Remove toast notifications for test-email success/failure (replaced by inline banner)
Co-Authored-By: Claude Sonnet 4.6 <[email protected]>
Co-Authored-By: Claude Sonnet 4.6 <[email protected]>
The card was disappearing because a successful send returned test_email_sent=true, which caused isCompleted to return true and filter the step out of visibleSteps immediately. Fix: isCompleted now returns false whenever emailStatus is not 'idle' (i.e. the user just sent an email and needs to see the result banner). Also added emailStatus to the stepsConfig useMemo dependency array. Co-Authored-By: Claude Sonnet 4.6 <[email protected]>
Previously the card disappeared after a successful send because test_email_sent=true caused isCompleted to return true. - isCompleted for sendTestEmail now only returns true when the step is explicitly skipped (skipped_steps includes 'send_test_email') - onSuccess merges only the email-status fields into the query cache, preserving test_email_sent so the redirect logic is unaffected Co-Authored-By: Claude Sonnet 4.6 <[email protected]>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
All Submissions:
Changes proposed in this Pull Request:
Closes # .
How to test the changes in this Pull Request:
Types of changes:
Other information:
Changelog entry