Skip to content

Comments

Add ability to disable comment edits and deletes#16

Merged
luisdalmolin merged 3 commits intomainfrom
feature/disable-comment-actions
Mar 17, 2025
Merged

Add ability to disable comment edits and deletes#16
luisdalmolin merged 3 commits intomainfrom
feature/disable-comment-actions

Conversation

@luisdalmolin
Copy link
Member

Summary

  • Added config options to disable comment editing and deletion
  • Added plugin configuration methods disallowEdits() and disallowDeletes()
  • Updated Comment.php to respect these settings
  • Includes tests and documentation updates

Test plan

  • Run all tests: ./vendor/bin/pest
  • Verify functionality by creating a new plugin with disallowEdits() and disallowDeletes() in a Laravel application

🤖 Generated with Claude Code

luisdalmolin and others added 3 commits March 10, 2025 07:41
- Added config options to disable comment editing and deletion
- Added plugin configuration methods disallowEdits() and disallowDeletes()
- Updated Comment.php to respect these settings
- Added tests to verify functionality
- Updated documentation

🤖 Generated with [Claude Code](https://claude.ai/code)
Co-Authored-By: Claude <noreply@anthropic.com>
- Updated disallowEdits and disallowDeletes methods to call Config class immediately
- This ensures settings take effect as soon as the methods are called
- Previously settings were only applied during plugin registration

🤖 Generated with [Claude Code](https://claude.ai/code)
Co-Authored-By: Claude <noreply@anthropic.com>
- Remove redundant class variables for edit/delete settings
- Empty register() method since config is set directly
- Simplified implementation with the same functionality

🤖 Generated with [Claude Code](https://claude.ai/code)
Co-Authored-By: Claude <noreply@anthropic.com>
@luisdalmolin luisdalmolin merged commit f541f59 into main Mar 17, 2025
4 checks passed
@luisdalmolin luisdalmolin deleted the feature/disable-comment-actions branch March 17, 2025 10:50
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.

1 participant