-
Notifications
You must be signed in to change notification settings - Fork 126
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
Update Laravel version to 8.0 #61
Conversation
- Packages updated - Exceptions in Handler.php changed to Throwable - String and array helpers updated to use Str and Arr in laravel/helpers - Email verification resending link updated to use post method - Changes to AppServiceProvider to continue using Bootstrap
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good-- only thing is possibly allowing doctrine/dbal to update, but I leave that at your discretion given the particulars of getting packages to cooperate with each other
composer.json
Outdated
@@ -8,32 +8,34 @@ | |||
], | |||
"license": "MIT", | |||
"require": { | |||
"php": "^7.1.3", | |||
"php": "^7.3", | |||
"boxconnect/deviant-php": "dev-master", | |||
"doctrine/dbal": "2.10.4", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This can I think safely be allowed to upgrade at this point, given that the issue was newer versions requiring php 7.2
Will the new Throwable need to be used for every instance of Exception currently used, across Extensions etc? |
Laravel version update
Bug fixes