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

Use config #224

Merged
merged 5 commits into from
Jan 8, 2024
Merged

Use config #224

merged 5 commits into from
Jan 8, 2024

Conversation

atmonshi
Copy link
Member

@atmonshi atmonshi commented Jan 5, 2024

using config in FE related stuff
this is important if you have multiple panels, also it's not necessary to use the BlotPlugin for none filament stuff

Copy link

what-the-diff bot commented Jan 5, 2024

PR Summary

  • Updated Testing Environment
    Adjusted the testing environment specifications in .github/workflows/run-tests.yml. Now, the testing will be performed only on PHP version 8.1 and Laravel version 10.*.

  • Refactored BoltPlugin Class References
    The LaraZeus\Bolt\BoltPlugin class has been replaced in numerous areas of the codebase, using config() calls instead. These adjustments have occurred in the following sections:

    • Forms and Responses Management
      The FillForms.php, ListEntries.php, ShowEntry.php, Response.php files now fetch configurations more effectively, promoting better app functionality.

    • Categories and Fields Management
      The files ListForms.php, Category.php, Field.php, and Section.php now use a better method to fetch category, form and field related configurations respectively.

    • Code Clean Up
      Some redundant/superfluous code has been removed from ListForms.php and Form.php enhancing the code cleanliness and performance.

This enhances the maintainability of the codebase by relying on configuration instead of directly using the class. The changes promote a more efficient code structure.

@atmonshi atmonshi changed the base branch from 2.x to 3.x January 8, 2024 18:32
@atmonshi atmonshi merged commit be22fe9 into 3.x Jan 8, 2024
@atmonshi atmonshi deleted the use-config branch January 8, 2024 18:34
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