Repository navigation
Upgrade to Laravel 13 and move from Laravel Mix to Vite - #132
Merged
Merged
Conversation
Laravel 13 - laravel/framework ^13.0 (13.39.0), with minor package updates - Filament panel uses PreventRequestForgery, the new name of VerifyCsrfToken - Providers move to bootstrap/providers.php; api and broadcast channel routes load from bootstrap/app.php - Remove the pre-Laravel 11 Auth, Broadcast, Event and Route service providers and the unused app/Http/Middleware classes; the Super Admin gate moves to AppServiceProvider and Fortify's home is '/admin' - Fixes duplicate verification emails: SendEmailVerificationNotification was registered on Registered twice (the old EventServiceProvider and the framework's) Vite - vite.config.js replaces webpack.mix.js; yarn dev / yarn build - Blade loads assets with @Vite and Vite::asset(); images are versioned in public/build - Isotope and imagesLoaded come from npm as jQuery plugins instead of vendored bundles; lodash and the separate postcss dependency are removed - The reCAPTCHA script waits for DOMContentLoaded, since the module script is deferred - Removes laravel-mix/webpack and with them every open npm security alert (webpack-dev-server, webpack-dev-middleware, uuid, elliptic); yarn audit is clean Deploy - CI builds with yarn build and checks for the Vite manifest - public/build ships with each release; public/css, js, fonts, images, svg and mix-manifest.json are no longer shared (the database has no links to them) Supersedes Dependabot #130 (laravel/framework 13.39.0) and #131 (axios 1.21.0, postcss).
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Upgrades Digitization Academy to Laravel 13 and replaces Laravel Mix with Vite.
Laravel 13
laravel/framework^13.0 (13.39.0), plus minor package updates. Every package resolves on 13.PreventRequestForgery, Laravel 13's new name forVerifyCsrfToken.bootstrap/providers.phpinstead ofconfig/app.php.apiand broadcast channel routes load frombootstrap/app.php.app/Http/Middlewareclasses are removed.AppServiceProvider, and Fortify'shomeis'/admin'.SendEmailVerificationNotificationwas registered twice onRegistered(the oldEventServiceProviderplus the framework's), so new users got two verification emails. It's registered once now.config/session.phpandconfig/cache.phpdon't setserializationorserializable_classes, so Laravel 13 keeps the old defaults (no forced logouts). The response cache stores JSON.Vite
vite.config.jsreplaceswebpack.mix.js. The scripts are nowyarn devandyarn build.@viteandVite::asset(). All images underresources/imagesare versioned inpublic/build.isotope-layout,imagesloaded,jquery-bridget) as jQuery plugins, because Vite 8 can't bundle the vendored minified copies.lodash(unused) and the separatepostcssdependency are removed.DOMContentLoaded, because Vite's module script is deferred.Dependencies and security
yarn.lockis down to 82 packages;yarn auditandcomposer auditare clean.Deploy
yarn buildand fails ifpublic/build/manifest.jsonis missing.public/buildships with each release instead of being shared, so a rollback also rolls back the assets.public/css,js,fonts,images,svgandmix-manifest.jsonare no longer shared; Filament republishes its own files on each deploy. Neither database has links to/images/,/css/,/js/,/fonts/or/svg/.shared/can be deleted.Testing
config:cache,route:cache,view:cacheandevent:cacheall succeed.yarn devhot reload.When this reaches
main, put[major]in the merge commit message so the release gets a major version bump.