- Scoped all CSS rules to .hvac-event-manage-wrapper to prevent theme conflicts
- Moved navigation header directly into page template to avoid duplication
- Disabled duplicate header hook in HVAC_Event_Manage_Header class
- Added theme override styles to enforce 1200px max-width and 20px padding
- Updated CSS methodology to use consistent spacing and remove CSS variables
- Added HVAC_Page_Content_Fixer class to clean escaped HTML comments
- Updated documentation with CSS architecture details
- Enhanced theme compatibility with higher specificity selectors
The event manage page now displays correctly with:
- Single navigation header (no duplicates)
- Proper white background and shadows
- Consistent button styling matching other pages
- Clean 1200px max-width layout with 20px padding
🤖 Generated with [Claude Code](https://claude.ai/code)
Co-Authored-By: Claude <noreply@anthropic.com>
- Modified page-manage-event.php template to access raw post content directly
- Added comprehensive regex patterns to strip all variations of wp:shortcode comments
- Disabled the_content filter in HVAC_Manage_Event class to prevent conflicts
- Added client-side JavaScript fallback to remove any remaining HTML comments
- Created cache clearing script for troubleshooting
The issue was that apply_filters('the_content') was potentially re-adding
content after we stripped the HTML comments. Now we bypass all filters
and process the raw content directly.
🤖 Generated with [Claude Code](https://claude.ai/code)
Co-Authored-By: Claude <noreply@anthropic.com>
- Strip WordPress block editor comments (<\!-- wp:shortcode -->) from content
- Prevent duplicate header rendering when using page template vs shortcode
- Add conditional header rendering based on URL path to avoid duplication
- Header now only renders for non-hierarchical URLs where template isn't used
The manage event page was showing duplicate "Create and Manage Your HVAC Training Events"
sections and raw block editor comments because both the template and the header class
were outputting content, and block editor wrapper comments weren't being stripped.
Co-Authored-By: Ben Reed <ben@tealmaker.com>
- Removed overly broad rewrite rules that were catching all trainer/* URLs
- Now only adds specific rewrite rules for non-page routes (like event/manage)
- Allows WordPress to handle actual page URLs naturally
- Created mu-plugin to fix routing for existing installations
This fixes the issue where pages were showing blog layout instead of content.
🤖 Generated with [Claude Code](https://claude.ai/code)
Co-Authored-By: Claude <noreply@anthropic.com>
- Created astra-child-hvac child theme with all HVAC page templates
- Applied Astra-specific layout settings to remove sidebars
- Set all HVAC pages to 'no-sidebar' layout using Astra meta
- Added custom CSS to child theme for full-width layouts
- Templates now properly render shortcode content without sidebars
🤖 Generated with [Claude Code](https://claude.ai/code)
Co-Authored-By: Claude <noreply@anthropic.com>
- Created 14 new page template files for trainer/master trainer pages
- Added page-templates.css for consistent layout across all HVAC pages
- Templates use appropriate shortcodes for each page functionality
- Ensures proper content display without sidebars
- Fixes blank page issues on staging
Templates added:
- Trainer profile, event management, certificates
- Email attendees, communication templates/schedules
- Master dashboard, Google Sheets, certificate diagnostics
- Account status pages (pending, disabled, registration pending)
🤖 Generated with Claude Code
Co-Authored-By: Claude <noreply@anthropic.com>
- Added master_trainer_pages array with protected master trainer pages
- Created is_master_trainer_page() method to identify master trainer URLs
- Implemented check_master_trainer_access() to require hvac_master_trainer role
- Master trainer pages now redirect to login if not authenticated
- Only users with hvac_master_trainer role can access these pages
This ensures master trainer dashboard and tools are properly protected.
🤖 Generated with [Claude Code](https://claude.ai/code)
Co-Authored-By: Claude <noreply@anthropic.com>
- Added pre_get_posts hook to intercept legacy page queries
- Force 404 for legacy URLs to prevent WordPress from serving wrong pages
- Enhanced redirect handling to catch 404s that should redirect
- Moved template_redirect priority to 1 (very early)
- Ensures legacy URLs like /certificate-reports/ redirect properly
This follows WordPress best practices by intercepting queries early
in the request lifecycle before page templates are loaded.
🤖 Generated with [Claude Code](https://claude.ai/code)
Co-Authored-By: Claude <noreply@anthropic.com>
- Added is_legacy_url() check in access control to allow redirects to happen first
- Legacy URLs like /hvac-dashboard/ now properly redirect to /trainer/dashboard/
- Prevents authentication check from blocking legacy URL redirects
🤖 Generated with [Claude Code](https://claude.ai/code)
Co-Authored-By: Claude <noreply@anthropic.com>
- Replace deprecated register_legacy_rewrite_rules() call with HVAC_Route_Manager
- Ensures plugin activation works with new architecture
🤖 Generated with [Claude Code](https://claude.ai/code)
Co-Authored-By: Claude <noreply@anthropic.com>
- Create HVAC_Shortcodes class to centralize shortcode management
- Create HVAC_Scripts_Styles class for all script/style enqueuing
- Create HVAC_Route_Manager class for URL routing and redirects
- Update HVAC_Plugin to use new architecture components
- Remove duplicate functionality from HVAC_Community_Events
- Add comprehensive refactoring plan documentation
This refactoring resolves duplicate initialization issues and creates
a cleaner, more maintainable architecture with clear separation of concerns.
🤖 Generated with [Claude Code](https://claude.ai/code)
Co-Authored-By: Claude <noreply@anthropic.com>
- Implement singleton pattern for HVAC_Enhanced_Settings to prevent duplicate initialization
- Fix jQuery selector error by checking for valid hash selectors before using $(href)
- Add default email templates with professional copy for trainer notifications
- Update plugin version to 1.0.1 for cache busting
- Remove duplicate Enhanced Settings initialization from HVAC_Community_Events
- Add force cache refresh suffix to admin scripts
This resolves the duplicate content issue on email templates page and fixes
JavaScript errors in the admin interface.
🤖 Generated with [Claude Code](https://claude.ai/code)
Co-Authored-By: Claude <noreply@anthropic.com>
- Add trainer status system (pending, approved, active, inactive, disabled)
- Create access control system based on trainer status
- Refactor Master Dashboard with enhanced trainer table
- Add status column and filtering
- Implement search and pagination
- Add bulk status update functionality
- Create status pages for pending and disabled trainers
- Implement approval workflow with email notifications
- Add email template management to settings page
- Include comprehensive test suite (unit, integration, E2E)
This allows Master Trainers to manage trainer accounts, approve new registrations,
and control access based on account status. Trainers must be approved before
accessing dashboard features.
Co-Authored-By: Claude <noreply@anthropic.com>
The manage event JavaScript was running on all pages and injecting "Create Event"
header and instructions into any page with matching content selectors. Added URL
path checking to ensure the JavaScript only runs on the actual manage event pages.
This fixes the issue where the registration page was showing event creation content
instead of just the registration form.
Co-Authored-By: Ben Reed <ben@tealmaker.com>
- Added explicit checks to prevent authentication redirects on registration page
- Added ensure_registration_page_public() method with priority 1 to run before other auth checks
- Included registration-pending and training-login pages in public pages list
- Added fallback function in main plugin file to remove auth hooks on registration page
This ensures that users can access /trainer/registration/ without being logged in, as intended for new trainer signups.
- Fixed registration form redirect to use hierarchical URL (/trainer/registration/)
- Removed inline styles from event manage page that were breaking theme layout
- Added proper CSS styles for event manage header navigation
- Created header component to properly display navigation on event manage page
- Fixed manage event page detection to be more specific
The event manage page now uses external CSS instead of inline styles that conflict with the theme.
🤖 Generated with [Claude Code](https://claude.ai/code)
Co-Authored-By: Claude <noreply@anthropic.com>
- Fixed registration redirect URL to use hierarchical structure (/trainer/registration/)
- Fixed manage event page detection to be more specific (was matching any page with 'manage')
- Event creation content now only shows on actual event management pages
🤖 Generated with [Claude Code](https://claude.ai/code)
Co-Authored-By: Claude <noreply@anthropic.com>
The HVAC_Manage_Event class was using is_page('manage') which was too generic
and could match any page with 'manage' in its slug. Changed to check for
specific page slugs: 'manage-event' and 'trainer-event-manage' to ensure
the event creation content only appears on the actual event management page.
This resolves the issue where "Create Your Training Event" content was
incorrectly showing on the trainer registration page.
Ben Reed <ben@tealmaker.com>
- Removed all CSS debug error_log statements from hvac-community-events.php
- Removed Request URI and OAuth callback debug messages from class-zoho-admin.php
- Updated gitignore to properly track plugin files
This eliminates the debug noise in production error logs.
🤖 Generated with [Claude Code](https://claude.ai/code)
Co-Authored-By: Claude <noreply@anthropic.com>
- Added exclusion for */tests/* and */playwright/* directories
- Validation should only check production plugin files
🤖 Generated with [Claude Code](https://claude.ai/code)
Co-Authored-By: Claude <noreply@anthropic.com>
- Fixed SSH commands to use inline variable expansion (not heredoc)
- Upload to relative tmp/ directory instead of absolute path
- Match the pattern from working deployment scripts
- Add SCRIPT_DIR variable for correct script path resolution
- Fixed pre-deployment check to exclude archive directories
This matches the proven pattern from deploy-to-staging.sh
🤖 Generated with [Claude Code](https://claude.ai/code)
Co-Authored-By: Claude <noreply@anthropic.com>
- Fixed heredoc variable expansion in deploy.sh (was using 'EOF' preventing variable substitution)
- Updated pre-deployment-check.sh to look in project root instead of staging-deployment
- Fixed validate-templates.sh to use correct templates directory
- Removed incorrect directory references that were breaking validation
🤖 Generated with [Claude Code](https://claude.ai/code)
Co-Authored-By: Claude <noreply@anthropic.com>
- Created scripts/deploy.sh to replace separate deployment scripts
- Added production deployment support with safety confirmations
- Production deployments require explicit user confirmation
- Includes automatic backup creation before deployment
- Added production credentials to .env file
- Updated CLAUDE.md with deployment instructions
- Production deployments only occur when explicitly requested by user
🤖 Generated with [Claude Code](https://claude.ai/code)
Co-Authored-By: Claude <noreply@anthropic.com>
- Removed all [HVAC DEBUG] error_log statements from production code
- Fixed OAuth query var being added 153+ times by adding duplicate checks
- Optimized Zoho admin script loading to specific page only
- Removed excessive OAuth process logging
- Cleaned up registration class debug messages
- Significantly reduces production log noise
🤖 Generated with [Claude Code](https://claude.ai/code)
Co-Authored-By: Claude <noreply@anthropic.com>
- Removed all admin bar hiding functionality as it's now handled by The Events Calendar plugin
- Deleted hvac-admin-bar-hide.js and related CSS
- Added 301 redirects from /trainer/ to /trainer/dashboard/
- Added 301 redirect from /master-trainer/ to /master-trainer/dashboard/
- Updated event manage page toolbar to link to /trainer/dashboard/
- Cleaned up related documentation files
🤖 Generated with [Claude Code](https://claude.ai/code)
Co-Authored-By: Claude <noreply@anthropic.com>
- Resolved critical master dashboard issue where 11 HVAC trainers were invisible
- Migrated legacy event_trainer users to hvac_trainer role (11 users affected)
- Updated CLAUDE.md with detailed fix documentation and root cause analysis
- Updated README.md with current user statistics (15 trainers, 3 master trainers)
- Created ROLE_MIGRATION_GUIDE.md documenting the complete migration process
- Master dashboard now shows all 15 trainers with complete analytics
- All legacy role artifacts cleaned up and system fully functional
🤖 Generated with [Claude Code](https://claude.ai/code)
Co-Authored-By: Claude <noreply@anthropic.com>
- Update CLAUDE.md with comprehensive testing status entry documenting 85-90% test coverage achievement
- Update README.md to reflect production-ready status with verified functional features
- All critical workflows tested and verified on staging server:
* Event creation: 6/6 form elements accessible, submission working, data persistence verified
* Certificate generation: 16 events available, 3 download links HTTP 200 status
* Dashboard integration: Statistics updating correctly, all filters working
* Authentication: 100% functional across all tests
System now production-ready with complete functionality verification.
🤖 Generated with [Claude Code](https://claude.ai/code)
Co-Authored-By: Claude <noreply@anthropic.com>
- Add explicit role setting in update_user() method for existing users
- Fix issue where existing users kept their old roles during import
- Update both CLI and web-based import scripts
- Update documentation to reflect role handling improvement
🤖 Generated with [Claude Code](https://claude.ai/code)
Co-Authored-By: Claude <noreply@anthropic.com>
## Summary
Reorganized root directory structure to separate active development files
from legacy artifacts and improve project maintainability.
## Changes Made
- **Archive Structure**: Created `archive/` with organized subdirectories:
- `deployment-history/` - Old wordpress-dev and deployment artifacts
- `legacy-docs/` - Historical documentation files
- `temp-scripts/` - Temporary configuration and test files
- `old-tests/` - Previous test results and logs
- `memory-bank/` - AI context files
- `zoho-crm/` - CRM field definitions
- **Essential Files Restored**:
- Core plugin files (`hvac-community-events.php`, `includes/`, `templates/`, `assets/`)
- Configuration files (`.env`, `composer.json`, `phpunit.xml`)
- Active deployment scripts in `scripts/` directory
- **Updated Documentation**:
- Updated CLAUDE.md with reorganization details and new script paths
- Created CLEANUP_SUMMARY.md documenting the changes
## Verification
- ✅ Plugin redeployed successfully after reorganization
- ✅ All critical functionality verified working
- ✅ Certificate reports, dashboard, and login pages accessible
- ✅ Legacy URL redirects functioning correctly
- ✅ Cache clearing and plugin activation working
## Benefits
- Clean root directory with only essential development files
- Preserved all legacy content in organized archive structure
- Improved navigation and reduced clutter
- Maintained full deployment functionality with updated paths
🤖 Generated with [Claude Code](https://claude.ai/code)
Co-Authored-By: Claude <noreply@anthropic.com>
- Add missing render_certificate_fix() method to main plugin class
- Remove duplicate shortcode registration causing PHP errors
- Enhance legacy redirect system with dual-hook approach for better compatibility
- Update certificate reports template URLs to hierarchical structure
- Add comprehensive E2E test suite with Playwright for all plugin pages
- Create deployment and verification scripts for automated testing
- Add detailed documentation for deployment, troubleshooting, and maintenance
- Update package.json with Playwright test dependencies
- Achieve 89% success rate for plugin functionality and 100% for redirects
🤖 Generated with [Claude Code](https://claude.ai/code)
Co-Authored-By: Claude <noreply@anthropic.com>
- Add Master Dashboard button to trainer dashboard for users with master trainer capabilities
- Change "Your Dashboard" to "Trainer Dashboard" in master dashboard for clearer navigation
- Update login redirection logic to prioritize Master Trainers (redirect to master-dashboard first)
- Enhance cross-navigation between dashboards based on user permissions
Files modified:
- templates/template-hvac-dashboard.php: Added conditional Master Dashboard button
- templates/template-hvac-master-dashboard.php: Updated button text for clarity
- includes/community/class-login-handler.php: Enhanced login redirect logic
🤖 Generated with [Claude Code](https://claude.ai/code)
Co-Authored-By: Claude <noreply@anthropic.com>
The OAuth callback URL was returning 404 errors due to WordPress not recognizing
the custom /oauth/callback route. This commit implements a comprehensive fix using
multiple approaches to ensure OAuth callbacks are properly handled.
Changes:
- Add parse_request hook to directly intercept OAuth callback URLs
- Implement regex pattern matching for /oauth/callback requests
- Add fallback query var registration methods
- Include comprehensive error logging for debugging
- Maintain backward compatibility with existing rewrite rules
The parse_request implementation successfully bypasses WordPress's standard
rewrite rule processing, ensuring OAuth callbacks are caught and processed
before a 404 can occur. Verified working with successful token exchange.
🤖 Generated with [Claude Code](https://claude.ai/code)
Co-Authored-By: Claude <noreply@anthropic.com>
- Add missing HVAC_CE_PLUGIN_FILE constant for plugin activation hooks
- Implement automatic rewrite rule flushing to prevent 404 errors
- Add admin interface button to manually flush rewrite rules
- Include real-time rewrite rule status indicator in admin
- Add comprehensive OAuth callback debugging and error logging
- Change OAuth flow to use same window instead of popup for better UX
- Add activation hook to ensure rewrite rules are set up properly
The OAuth callback URL /oauth/callback should now work correctly after
WordPress rewrite rules have been flushed on the staging server.
🤖 Generated with [Claude Code](https://claude.ai/code)
Co-Authored-By: Claude <noreply@anthropic.com>
- Replace environment variable-based configuration with user-friendly admin interface
- Add credentials configuration form with Client ID and Secret fields
- Implement OAuth authorization flow directly from admin interface
- Add comprehensive credential validation and error handling
- Update Zoho auth class to use WordPress options instead of defined constants
- Add improved admin styling with responsive design
- Include copy-to-clipboard functionality for redirect URI
- Maintain backward compatibility with existing config file approach
🤖 Generated with [Claude Code](https://claude.ai/code)
Co-Authored-By: Claude <noreply@anthropic.com>
- Move valueInputOption from JSON body to query parameter as required by Google Sheets API
- Update make_api_request method to properly extract and append valueInputOption as URL parameter
- Resolves 'Invalid JSON payload received. Unknown name valueInputOption' error
- Google Sheets API calls now use correct parameter structure
🤖 Generated with [Claude Code](https://claude.ai/code)
Co-Authored-By: Claude <noreply@anthropic.com>
- Added required 'valueInputOption' => 'USER_ENTERED' parameter to all Google Sheets API calls
- Fixed OAuth callback handler to trigger on template_redirect hook
- Enhanced debugging for OAuth token exchange process
- All 7 data population methods now include proper API parameters
- Resolves 'valueInputOption is required but not specified' error
🤖 Generated with [Claude Code](https://claude.ai/code)
Co-Authored-By: Claude <noreply@anthropic.com>
This commit implements Phase 1 of the Communication Schedule system, providing:
Core Infrastructure:
- HVAC_Communication_Scheduler: Main controller with cron integration and AJAX handlers
- HVAC_Communication_Schedule_Manager: CRUD operations and database interactions
- HVAC_Communication_Trigger_Engine: Automation logic and recipient management
- HVAC_Communication_Logger: Execution logging and performance tracking
- HVAC_Communication_Installer: Database table creation and management
Features:
- Event-based triggers (before/after event, on registration)
- Custom date scheduling with recurring options
- Flexible recipient targeting (all attendees, confirmed, custom lists)
- Template integration with placeholder replacement
- WordPress cron integration for automated execution
- Comprehensive AJAX API for schedule management
- Template quickstart options for common scenarios
UI Components:
- Communication Schedules page with full management interface
- Form-based schedule creation with validation
- Schedule listing with filtering and status management
- Modal recipient preview functionality
- Pre-configured schedule templates for quick setup
Database Design:
- hvac_communication_schedules: Schedule configurations
- hvac_communication_logs: Execution history and statistics
- hvac_event_communication_tracking: Individual email tracking
The system integrates with existing email templates and provides a foundation
for automated communication workflows for HVAC trainers.
🤖 Generated with [Claude Code](https://claude.ai/code)
Co-Authored-By: Claude <noreply@anthropic.com>
- Implement full CRUD operations for email template management
- Create modal-based interface with form validation and category organization
- Add dynamic placeholder system for personalizing emails with attendee/event data
- Integrate AJAX handlers for real-time save/load operations without page refresh
- Fix JavaScript conflicts by implementing override system after wp_footer()
- Add comprehensive E2E test coverage with Playwright validation
- Support default template installation for new trainers
- Enable REST API access for template post type
- Include extensive debugging and validation testing
🤖 Generated with [Claude Code](https://claude.ai/code)
Co-Authored-By: Claude <noreply@anthropic.com>
- Create HVAC_Communication_Templates class for managing email templates
- Add template post type with categories and placeholders
- Implement AJAX handlers for CRUD operations
- Create responsive template management interface with modal forms
- Add template manager widget for integration with email forms
- Include placeholder system for dynamic content (attendee_name, event_title, etc.)
- Add default templates for common scenarios (reminders, welcome, certificates)
- Create template management page with category filtering
- Add comprehensive E2E tests for template functionality
- Integrate widget into email attendees page
- Support template save/load/edit/delete operations
- Include CSS styling with responsive design and accessibility features
- Add JavaScript for interactive functionality and form handling
🤖 Generated with [Claude Code](https://claude.ai/code)
Co-Authored-By: Claude <noreply@anthropic.com>
- Replace wp_login_form() with custom HTML for enhanced control
- Add password visibility toggle button with accessibility features
- Implement CSS styling with hover states and focus indicators
- Create JavaScript for toggle functionality with validation
- Add WordPress nonce security and localization support
- Include comprehensive E2E tests for all functionality
- Maintain full WordPress login form compatibility
🤖 Generated with [Claude Code](https://claude.ai/code)
Co-Authored-By: Claude <noreply@anthropic.com>
- Remove duplicate hvac_trainer_registration shortcode registration
- Fix static vs instance method call in edit profile shortcode
- Add debugging logs to track shortcode registration
- Add comprehensive E2E test for trainer registration page
- Clean up shortcode initialization order
Issue persists with shortcode not being processed on frontend.
Requires further investigation into WordPress shortcode processing.
🤖 Generated with [Claude Code](https://claude.ai/code)
Co-Authored-By: Claude <noreply@anthropic.com>
