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

Fix category tenant #269

Merged
merged 2 commits into from
Apr 4, 2024
Merged

Fix category tenant #269

merged 2 commits into from
Apr 4, 2024

Conversation

atmonshi
Copy link
Member

@atmonshi atmonshi commented Apr 4, 2024

No description provided.

Copy link

what-the-diff bot commented Apr 4, 2024

PR Summary

  • Ignoring Non-critical Files in Git
    The .phpunit.cache/test-results file has been added to the .gitignore. This means Git will ignore this file, reducing noise for developers when detecting changes in the project.

  • Updated Test Results
    The .phpunit.cache/test-results file has been updated to contain the latest test results. This doesn't affect the application, but helps to keep track of unit testing results.

  • Changes to src/Concerns/Schemata.php File

    • Namespaces Import: New namespaces, Filament\Facades\Filament and Illuminate\Database\Eloquent\Builder have been added to interact better with the Filament package and Laravel's query builder.
    • Use of Filament Facade: The code now uses the Filament facade (a class to provide a simplified interface), which allows easier interactions with the Filament functionalities.
    • Improvements to the relationship Method: This method has been modified to include a new functionality. It now can react based on whether Filament has a certain 'tenant' set. The adjusted query helps in achieving more relevant results.

@atmonshi atmonshi merged commit f7c0b8b into 3.x Apr 4, 2024
5 checks passed
@atmonshi atmonshi deleted the fix-category-tenant branch April 4, 2024 04:16
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