okaufmann / laravel-web-money-manager-ex

5 stars 1 forks source link

Laravel 5.7 Shift #97

Closed okaufmann closed 5 years ago

okaufmann commented 5 years ago

This pull request includes the changes for upgrading to Laravel 5.7. Feel free to commit any additional changes to the shift-10659 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.

okaufmann commented 5 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:

okaufmann commented 5 years ago

⚠ Laravel 5.7 changed the environment variable QUEUE_DRIVER to QUEUE_CONNECTION. You should review your server configuration or .env files to see if they need to be changed.

okaufmann commented 5 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.

okaufmann commented 5 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.

okaufmann commented 5 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.

okaufmann commented 5 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.