Include Entire Laravel Framework
gagegogan opened this issue · comments
I've been having trouble integrating certain Laravel packages like InertiaJS and Ziggy due to the hard requirements that Themosis has on requiring Laravel. I'm suggesting that we remove the individual package requirements of individual Laravel packages and just require the entire framework. Additionally, I'm suggesting we upgrade ragonmantank/cron-expression to 3.0 (this is a Laravel 8 requirement). I've created a fork and did this. And I haven't noticed any problems. I also have a composer script to rename the Laravel __() function but I'm not sure where you would like me to place it.
I was also having the same problem before. One solution that I did before was to copy the laravel foundation directory to themosis. I was able to make it work but it was too much of a hassle to do so. I eventually found another solutions which already support Laravel 8. Its called Laraish. I was able to integrate inertia js with React. I do hope that Themosis will eventually have this feature because I still like the overall integration of Themosis to Wordpress compared to Laraish.
We won't add the entire laravel framework into Themosis as is as foundation code is different. InertiaJS and other recent libraries working with Laravel are not compatible because current stable release of Themosis is targeting Illuminate package version 5.x where it needs at least version 7 or 8 to work.
The master branch is holding code to support Laravel 8.x so this should resolve normally on next release.
@jlambe I applied these changes too the master branch. There were no issues. There is no reason to my knowledge that should break it
Additionally the master branch will not support it because of the cron-express package being 2.0 not 3.0
@armycoder What's the blocking part exactly ? Is it just upgrading the cron package ?
I believe so.
I'll make sure to have the dependencies updated and then we'll test how InertiaJS behave with Themosis.