From 2a75b6808daeb3b73de8003a559c6ccb6b37b9b3 Mon Sep 17 00:00:00 2001 From: bengizmo Date: Fri, 23 May 2025 16:27:14 -0300 Subject: [PATCH] Replace alert() calls with HVACToast system in certificate and Zoho admin JavaScript MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Updated hvac-certificate-actions.js and zoho-admin.js to use the modern toast notification system instead of browser alerts, with fallback to alert() if HVACToast is not available. This provides a better user experience with consistent styling and non-blocking notifications. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude --- .../assets/js/hvac-certificate-actions.js | 77 ++++++++++++++++--- .../assets/js/zoho-admin.js | 21 ++++- 2 files changed, 84 insertions(+), 14 deletions(-) diff --git a/wordpress-dev/wordpress/wp-content/plugins/hvac-community-events/assets/js/hvac-certificate-actions.js b/wordpress-dev/wordpress/wp-content/plugins/hvac-community-events/assets/js/hvac-certificate-actions.js index caef97ef..054af1c1 100644 --- a/wordpress-dev/wordpress/wp-content/plugins/hvac-community-events/assets/js/hvac-certificate-actions.js +++ b/wordpress-dev/wordpress/wp-content/plugins/hvac-community-events/assets/js/hvac-certificate-actions.js @@ -59,12 +59,22 @@ window.open(response.data.url, '_blank'); } } else { - alert(response.data.message || 'Error: Failed to get certificate URL'); + // Use toast notification instead of alert + if (window.HVACToast) { + HVACToast.error(response.data.message || 'Failed to get certificate URL'); + } else { + alert(response.data.message || 'Error: Failed to get certificate URL'); + } } }, error: function() { $button.prop('disabled', false).removeClass('hvac-loading'); - alert('Error: Failed to connect to server'); + // Use toast notification instead of alert + if (window.HVACToast) { + HVACToast.error('Failed to connect to server'); + } else { + alert('Error: Failed to connect to server'); + } } }); }, @@ -126,18 +136,33 @@ $button.prop('disabled', false).removeClass('hvac-loading'); if (response.success) { - alert(response.data.message || 'Certificate sent successfully'); + // Use toast notification instead of alert + if (window.HVACToast) { + HVACToast.success(response.data.message || 'Certificate sent successfully'); + } else { + alert(response.data.message || 'Certificate sent successfully'); + } // Update UI to indicate certificate has been emailed const $row = $button.closest('tr'); $row.find('.hvac-certificate-emailed').text('Yes'); } else { - alert(response.data.message || 'Error: Failed to send certificate'); + // Use toast notification instead of alert + if (window.HVACToast) { + HVACToast.error(response.data.message || 'Failed to send certificate'); + } else { + alert(response.data.message || 'Error: Failed to send certificate'); + } } }, error: function() { $button.prop('disabled', false).removeClass('hvac-loading'); - alert('Error: Failed to connect to server'); + // Use toast notification instead of alert + if (window.HVACToast) { + HVACToast.error('Failed to connect to server'); + } else { + alert('Error: Failed to connect to server'); + } } }); }, @@ -176,7 +201,12 @@ $button.prop('disabled', false).removeClass('hvac-loading'); if (response.success) { - alert(response.data.message || 'Certificate revoked successfully'); + // Use toast notification instead of alert + if (window.HVACToast) { + HVACToast.success(response.data.message || 'Certificate revoked successfully'); + } else { + alert(response.data.message || 'Certificate revoked successfully'); + } // Update UI to indicate certificate has been revoked const $row = $button.closest('tr'); @@ -188,12 +218,22 @@ $button.hide(); $row.find('.hvac-unrevoke-certificate').show(); } else { - alert(response.data.message || 'Error: Failed to revoke certificate'); + // Use toast notification instead of alert + if (window.HVACToast) { + HVACToast.error(response.data.message || 'Failed to revoke certificate'); + } else { + alert(response.data.message || 'Error: Failed to revoke certificate'); + } } }, error: function() { $button.prop('disabled', false).removeClass('hvac-loading'); - alert('Error: Failed to connect to server'); + // Use toast notification instead of alert + if (window.HVACToast) { + HVACToast.error('Failed to connect to server'); + } else { + alert('Error: Failed to connect to server'); + } } }); } @@ -254,11 +294,21 @@ GenerateCertificates.renderAttendees(response.data.attendees, response.data.event_title); $('#step-select-attendees').show(); } else { - alert('Error: ' + response.data.message); + // Use toast notification instead of alert + if (window.HVACToast) { + HVACToast.error(response.data.message); + } else { + alert('Error: ' + response.data.message); + } } }, error: function() { - alert('Failed to load attendees. Please try again.'); + // Use toast notification instead of alert + if (window.HVACToast) { + HVACToast.error('Failed to load attendees. Please try again.'); + } else { + alert('Failed to load attendees. Please try again.'); + } }, complete: function() { $('#attendees-loading').hide(); @@ -444,7 +494,12 @@ const attendeeName = $button.data('attendee'); if (!url) { - alert('Preview URL not available'); + // Use toast notification instead of alert + if (window.HVACToast) { + HVACToast.error('Preview URL not available'); + } else { + alert('Preview URL not available'); + } return; } diff --git a/wordpress-dev/wordpress/wp-content/plugins/hvac-community-events/assets/js/zoho-admin.js b/wordpress-dev/wordpress/wp-content/plugins/hvac-community-events/assets/js/zoho-admin.js index 8358a9e9..f1cb819f 100644 --- a/wordpress-dev/wordpress/wp-content/plugins/hvac-community-events/assets/js/zoho-admin.js +++ b/wordpress-dev/wordpress/wp-content/plugins/hvac-community-events/assets/js/zoho-admin.js @@ -233,13 +233,28 @@ jQuery(document).ready(function($) { }, success: function(response) { if (response.success) { - alert('Settings saved successfully!'); + // Use toast notification instead of alert + if (window.HVACToast) { + HVACToast.success('Settings saved successfully!'); + } else { + alert('Settings saved successfully!'); + } } else { - alert('Error saving settings: ' + response.data.message); + // Use toast notification instead of alert + if (window.HVACToast) { + HVACToast.error('Error saving settings: ' + response.data.message); + } else { + alert('Error saving settings: ' + response.data.message); + } } }, error: function() { - alert('Error saving settings'); + // Use toast notification instead of alert + if (window.HVACToast) { + HVACToast.error('Error saving settings'); + } else { + alert('Error saving settings'); + } }, complete: function() { $button.prop('disabled', false).text('Save Settings');