Merge pull request #149 from mailchimp/fix/147

Add a confirmation prompt before logging out the user.

Author: vikrampm1

tests/cypress/e2e/logout.test.js

@@ -7,6 +7,12 @@ describe('Logout tests', () => {
 	it('Admin can logout', () => {
 		cy.visit('/wp-admin/admin.php?page=mailchimp_sf_options');
 		cy.get('#mailchimp_sf_oauth_connect').should('not.exist');
+
+		cy.on('window:confirm', (text) => {
+			expect(text).to.contains('Are you sure you want to log out?');
+			return true;
+		});
+
 		cy.get('input[value="Logout"]').click();
 
 		// connect to "Mailchimp" Account button should be visible.

views/setup_page.php

@@ -21,7 +21,7 @@
 			<td><h3><?php esc_html_e( 'Logged in as', 'mailchimp' ); ?>: <?php echo esc_html( $user['username'] ); ?></h3>
 			</td>
 			<td>
-				<form method="post" action="">
+				<form method="post" action="" onsubmit="return confirm('<?php echo esc_js( __( 'Are you sure you want to log out?', 'mailchimp' ) ); ?>');">
 					<input type="hidden" name="mcsf_action" value="logout"/>
 					<input type="submit" name="Submit" value="<?php esc_attr_e( 'Logout', 'mailchimp' ); ?>" class="button button-secondary mailchimp-sf-button small" />
 					<?php wp_nonce_field( 'mc_logout', '_mcsf_nonce_action' ); ?>