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

Upgrade Laravel #354

Merged
merged 8 commits into from
Feb 27, 2021
Merged

Upgrade Laravel #354

merged 8 commits into from
Feb 27, 2021

Conversation

dshoreman
Copy link
Owner

@dshoreman dshoreman commented Feb 26, 2021

Updates laravel and all other Composer packages to their latest
version, including the new laravel/ui package for the Auth backend.

Default files have also been brought inline with laravel/laravel.

Commits split out from PR #353 when it became apparent that 8.1 support would be easier said than done.

Updates laravel to v7.30 and all other Composer packages to their latest
version, including the new laravel/ui package for the `Auth` backend.
@dshoreman dshoreman added the enhancement New feature or request label Feb 26, 2021
@dshoreman dshoreman self-assigned this Feb 26, 2021
@codecov
Copy link

codecov bot commented Feb 26, 2021

Codecov Report

Merging #354 (60d289d) into develop (5c557d8) will decrease coverage by 0.10%.
The diff coverage is 95.45%.

Impacted file tree graph

@@              Coverage Diff              @@
##             develop     #354      +/-   ##
=============================================
- Coverage      96.56%   96.46%   -0.11%     
- Complexity       365      366       +1     
=============================================
  Files             67       66       -1     
  Lines            962      962              
=============================================
- Hits             929      928       -1     
- Misses            33       34       +1     
Impacted Files Coverage Δ Complexity Δ
app/Console/Commands/Version.php 100.00% <ø> (ø) 1.00 <0.00> (ø)
app/Http/Middleware/Authenticate.php 66.66% <50.00%> (-33.34%) 2.00 <0.00> (+1.00) ⬇️
app/Http/Controllers/Auth/RegisterController.php 100.00% <100.00%> (ø) 4.00 <0.00> (ø)
app/Http/Middleware/RedirectIfAuthenticated.php 100.00% <100.00%> (ø) 4.00 <4.00> (+2.00)
app/Providers/RouteServiceProvider.php 100.00% <100.00%> (ø) 4.00 <3.00> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5c557d8...60d289d. Read the comment docs.

* Bumps composer dependencies
* Renames database/seeds to database/seeders
* Rewrites `UserFactory` inline with new class-based approach(!)
* Updates `User` model and `RequiresAuth` trait for new factory
These directories get moved or deleted when the test passes so, when
they fail and get left behind, we don't want the next run to break
purely as a result of the last one failing. By checking `is_dir` first,
we can continue silently and only create when it doesn't exist.
Renames the maintenance mode middleware in line with the default files
and updates the public index.php to use the cached file if it exists.

Also sorts the .gitignore alphabetically where maintenance.php is added.
Drops the `$namespace` property so we can use fully qualified classes in
our routes. As part of the updates to `RouteServiceProvider`, the rate
limiting code has also moved here from `Http\Kernel::middlewareGroups`.
* Updates default .env template
* Drops redis facade from config/app.php
* Sets same-site cookies to be 'lax' and updates CORS paths
* Adds 'ably' as a supported broadcaster (whatever ably is...)
* Adds an `unverified` factory state, although it may not stay
* Updates `RedirectIfAuthenticated` to support multiple guards
* Adds `uuid` field to `failed_jobs` table for batching support
* Prevents `current_password` from being flashed in exception handler
* Removes unused `report()` and `render()` exception handler methods
* Removes unused `bindings` alias from `Http\Kernel::$routeMiddleware`
* Removes unused `filesystems.cloud` config option
* Enables overriding of log level with `LOG_LEVEL` environment variable
* Adds `auth.password` and `validation.multiple_of` to lang files
@dshoreman dshoreman merged commit 4c54bfb into develop Feb 27, 2021
@dshoreman dshoreman deleted the laravel-upgrade branch February 27, 2021 20:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant