Skip to content

Conversation

@iamgerwin
Copy link
Owner

Summary

Cleanup and refactoring for production readiness - removing debug logs and improving compatibility.

Changes

  • Removed all console.log debug statements from FormField.vue
  • Improved FieldServiceProvider to handle non-Nova environments gracefully
  • Updated PHP version requirement to support PHP 8.3, 8.4, and 8.5
  • Updated dependency versions for better compatibility

Technical Details

  • Reduced JavaScript bundle size by ~2.5KB (from 27.2KB to 24.7KB)
  • Added class_exists check before registering Nova assets
  • Fixed PHPStan analysis errors
  • All tests passing

Test Results

  • PHPStan: No errors
  • Pest: 25 tests passing (61 assertions)

- Remove all console.log statements from FormField.vue for production
- Add class_exists check in FieldServiceProvider for non-Nova environments
- Update PHP version to support 8.3, 8.4, and 8.5
- Update collision to ^8.6 and pest to ^3.7.4 for compatibility
- Reduce JS bundle size by ~2.5KB
- Fix PHPStan errors and all tests passing
@iamgerwin iamgerwin merged commit 5099b91 into main Nov 25, 2025
5 checks passed
@iamgerwin iamgerwin deleted the chore/v1.0.16-cleanup-and-refactor branch November 25, 2025 05:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants