Conversation
PR Summary
|
CLAUDE.md
Outdated
| - `composer format` - Auto-fix PHP coding standards issues | ||
| - `composer test` - Run PHP unit tests | ||
| - `composer test:phpstan` - Run PHPStan static analysis | ||
| - `composer compat` - Check PHP compatibility |
There was a problem hiding this comment.
Need to update these as many use npm not composer to run.
Preview changes with PlaygroundYou can preview the recent changes for PR#1116 with the following PHP versions: PHP Version 8.3
PHP Version 7.4
Download Made with 💙 from GatherPress & a little bit of WordPress Playground. Changes will not persist between sessions. |
stephenerdelyi
left a comment
There was a problem hiding this comment.
looks great to me, @mauteri. I did my usual rounds of testing on this PR and didn't come across any gotchas
Description of the Change
Major Features
• New RSVP Token System: Enables anonymous users to manage RSVPs via email-delivered tokens without requiring site accounts
• Interactive RSVP Forms: Added AJAX-powered form submission with real-time validation and error handling
• Email Token Confirmation: Automated email workflow for token-based RSVP authentication
Core Functionality
• Open RSVP Management: Users can update RSVP status through tokenized links sent to their email
• Form Field Processing: Enhanced custom field validation and sanitization with schema-based processing
• Nonce Handling: Dynamic nonce management with automatic retry logic for expired tokens
Code Architecture
• Method Refactoring: Moved form-related methods from Rsvp_Setup to Rsvp_Form class for better organization
• Built-in Fields Management: Extracted hard-coded field lists into reusable class constants
• API Enhancements: Extended REST API endpoints to support token-based authentication
Developer Experience
• Comprehensive Testing: Added unit tests for new token system and form processing functionality
• Documentation Updates: Added CLAUDE.md for AI-assisted development and updated hook
documentation
• Code Quality: PHPStan fixes, linting improvements, and consistent code formatting across PHP and JavaScript files
Closes #1087 #1093
How to test the Change
Changelog Entry
Credits
Props @mauteri
Checklist: