stefanzweifel / screeenly

📸 Screenshot as a Service
https://secure.screeenly.com
MIT License
503 stars 103 forks source link

Laravel 5.7 Shift #173

Closed stefanzweifel closed 6 years ago

stefanzweifel commented 6 years ago

This pull request includes the changes for upgrading to Laravel 5.7. Feel free to commit any additional changes to the shift-9206 branch.

Before merging, you should:

If you want help with your upgrade, check out the Shift Human Services or join the Shifty Coders Slack room to get answers to all your Laravel questions.

stefanzweifel commented 6 years ago

❌ Shift could not upgrade the following configuration files since they differed from the default Laravel version. You will need to compare these configuration files against the Laravel 5.7 configuration files and merge any changes:

stefanzweifel commented 6 years ago

Route::redirect now returns a 302 HTTP status code redirect in Laravel 5.7. Previously it returned a 301. Shift changed these to the new Route::permanentRedirect to preserve your intended behavior. You may want to review these changes if you are using the Route facade outside of the routes folder or want to return a 301.

stefanzweifel commented 6 years ago

ℹ Laravel 5.7 removed the Blade "or" operator in favor of PHP's built-in ?? "null coalesce" operator. Shift automated this change, however, your application may still contain instances of or if you are using Blade syntax outside of the resources folder.

stefanzweifel commented 6 years ago

⚠ Laravel 5.7 was just released. As such, many packages may not yet support Laravel 5.7. Be sure to let the author know. Better yet, submit a pull request as this is often simply a change in their version constraints.

stefanzweifel commented 6 years ago

ℹ Laravel 5.7 removed the resources/assets folder in favor of placing its sub-folders directly under resources. Shift moved these for you automatically. However, you may need to update the paths configured in your frontend build tool (e.g. webpack.mix.js) to reflect this new structure.

While Shift encourages a fully upgraded application, this new structure is not required. If this change significantly impairs your frontend build process, you can revert this commit.

stefanzweifel commented 6 years ago

ℹ Laravel 5.7 now supports email verification when users register for your application. If you have implemented this feature yourself and would like to take advantage of this native functionality, the steps are below. For more details, review the documentation on Email Verification.

stefanzweifel commented 6 years ago

ℹ Shift noticed you have additional namespaces in your project. You should consider using the Consolidate Namespaces Shift to simplify your namespaces into the default Laravel App namespace.