Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add filament tools package #332

Merged
merged 1 commit into from
Oct 29, 2024
Merged

add filament tools package #332

merged 1 commit into from
Oct 29, 2024

Conversation

atmonshi
Copy link
Member

No description provided.

Copy link

what-the-diff bot commented Oct 29, 2024

PR Summary

  • Updated Project Dependencies
    The PR introduces a new project dependency ("lara-zeus/filament-plugin-tools": "^1.0"), added to our project's composer.json file. This change provides us access to more mature tools for building and managing our application.

  • Improved Badge Management
    Badge visibility checks have been updated across several resource files (CategoryResource, CollectionResource, FormResource), which are now utilizing the more direct self::class notation rather than referring to the previously defined Resources:: enum.

  • Refinement of Code Base
    Dropped unused and redundant elements were from the Configuration.php file, contributing to a more streamlined, less cluttered codebase.

  • Enhanced Understanding of Force Deletion Process
    The addition of comments to bypass lines of code during PHPStan analysis in certain models (Field, Form, Response, and Section) helps provide insight into what is ignored during the force deletion process, thus making it easier to understand for other team members.

  • Minor Code Refactoring
    Some code refactoring was undertaken in the Toggle.php and FieldsContract.php files for improved readability and maintainability, such as the addition of a type hint for $response.

@atmonshi atmonshi merged commit a1d5727 into 3.x Oct 29, 2024
6 checks passed
@atmonshi atmonshi deleted the use-tools branch October 29, 2024 19:31
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