Skip to content

fix using laravel alias instead of fqdn#164

Merged
coolsam726 merged 1 commit intosavannabits:mainfrom
LinusTebbe:remove-alias-usage
Jan 26, 2026
Merged

fix using laravel alias instead of fqdn#164
coolsam726 merged 1 commit intosavannabits:mainfrom
LinusTebbe:remove-alias-usage

Conversation

@LinusTebbe
Copy link
Contributor

In the laravel project I work on we completely disabled aliases, using the Config alias forces the developer of a project to include atleast that alias which, in my opinion, shouldn't be a decision forced through a package

I also took the liberty of adding an early return and removing an unused variable from the code I worked on

@what-the-diff
Copy link

what-the-diff bot commented Jan 24, 2026

✨ Exciting PR Summary ✨

  • Cleaner Code with Consistency
    To ensure the code is easier to read, we've brought a new teammate on board, our Config facade! This simplifies the way we access configuration settings, making the code neater and more straightforward to follow. Now, we'll use Config::get throughout our codebase for a uniform look. Consistency is key! 🔑

  • Efficient Directory Creation
    We've given our directory creation logic a facelift, embracing elegance and simplicity. No more needless block of code that just waits around: we're all about that "early return" lifestyle now. It also ensures that the pathways for our virtual content banks (Filament Pages, Resources, and Widgets) are always carved out, even if they didn't exist before. So, welcome to a smoother, uncluttered flow of the directory creation process. Enjoy the improved, less messy journey! 🚀

@coolsam726 coolsam726 merged commit 1ff654e into savannabits:main Jan 26, 2026
1 of 25 checks passed
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

Comments