• Resolved generosus

    (@generosus)


    Good Day,

    Every time we click the “Email Test” tab, we are taken to a brand new settings stable. The behavior or functionality of the main Settings table is broken.

    Details (Main Settings Table): https://prnt.sc/kdlA7fmd232U
    Details (Main Settings Table Behavior Broken): https://prnt.sc/cpLV_iy7CCp1

    The “Email Test” tab is the only tab that causes this to happen.

    To fix the above, we would like to recommend removing the “Email Test” tab from the main Settings table. The Tools section of your plugin should suffice to find the “Email Test” feature of your plugin.

    Thank you!

Viewing 9 replies - 1 through 9 (of 9 total)
  • Plugin Support Dake

    (@dakeg)

    Hi @generosus

    Thanks for reaching out to us and we apologise for the delay.

    This has actually been the default behaviour of our settings for quite some time now.

    The “Email Test” tab belongs to our Tools section as you have mentioned. “Email Test” is included in the settings menu for convenience purposes. This way someone can quickly hop into email tests/debugging if need be.

    Your request is still valid in that we can consider adjusting this behavior, so thank you for sharing your request. I will share this thread with out team.

    Thread Starter generosus

    (@generosus)

    Hey @dakeg,

    Thank you. As we agree, the “Email Test” tab belongs in the “Tools” section of your plugin, nowhere else.

    Looking forward to the correction at your next plugin update.

    Have a great week.

    Plugin Support Dake

    (@dakeg)

    Hi @generosus

    Thanks for getting back to us.

    I’d like to clarify that we haven’t confirmed that the adjustment will be made in our next release.

    As mentioned in my previous reply, “Email Test” is included in the general settings for convenience. We’ll need to consider the benefits of both options as users have already gotten used to “Email Test” being in both locations.

    Thread Starter generosus

    (@generosus)

    Hey @dakeg,

    OK, but still believe breaking the flow or functionality of the main setttings table is wrong.

    Please leave this topic open until a decision has been made.

    Thank you.

    Plugin Support Dake

    (@dakeg)

    Hi @generosus

    Again, while your point is definitely valid, we have chosen to continue using the current locations for now. However, if you’re willing the “Email Test” link can be removed from the settings menu with a little custom code:

    // Remove test tab link from the settings page menu in the WP Mail SMTP plugin.
    add_filter( 'wp_mail_smtp_admin_get_pages', function ( $pages ) {
    
    	unset( $pages['test'] );
    
    	return $pages;
    }, PHP_INT_MAX );

    You would need to use a code snippet plugin likeΒ WPCode because it will not work from the functions.php file. The filter applies earlier than the theme loads.

    Another benefit is that it will protect the snippet from updates.

    In case it helps, you can find a tutorial for using WPCode here.

    We would like to apologize that we have opted to go a different route for now, but we really do hope that the code snippet will suffice for now.

    Thread Starter generosus

    (@generosus)

    Hi @dakeg,

    That’s even better. Keeps everyone happy πŸ™‚

    Just installed and tested the code snippet. Works like a charm.

    Issue resolved. Thank you!

    • This reply was modified 2 years, 10 months ago by generosus.
    Plugin Support Dake

    (@dakeg)

    Hi @generosus

    You’re welcome, glad we could help πŸ™‚

    Thread Starter generosus

    (@generosus)

    Hey @dakeg,

    One last request …

    It would be great if you can update your Code Snippets Documentation to add the information contained in this topic.

    Once updated, you can refer future inquiries to the updated documentation link. It’s the right thing to do πŸ™‚

    Thank you!

    Plugin Support Dake

    (@dakeg)

    Hi @generosus

    Thank you for the suggestion, I have shared this thread with our documentation team for future use πŸ˜€

Viewing 9 replies - 1 through 9 (of 9 total)

The topic ‘Enhancement Request | Main Settings Table Behavior’ is closed to new replies.