[PR #3021] [MERGED] Laravel 8.x Shift #6115

New Issue

OVERLORD · 2026-02-05T10:24:51+03:00

OVERLORD commented

2026-02-05 10:24:51 +03:00

📋 Pull Request Information

Original PR: https://github.com/BookStackApp/BookStack/pull/3021
Author: @ssddanbrown
Created: 10/28/2021
Status: ✅ Merged
Merged: 10/30/2021
Merged by: @ssddanbrown

Base: laravel_8_upgrade ← Head: shift-51221

📝 Commits (10+)

5b51039 Apply Laravel coding style
ad2c749 Shift exception handler
6950e6d Shift HTTP kernel and middleware
b5db904 Shift service providers
eb6a2c7 Convert options array to fluent methods
7dbdaca Shift to class based routes
b3898ac Shift console routes
01dd2b7 Ignore temporary framework files
3cd292a Shift to class based factories
5604965 Namespace seeders

📊 Changes

86 files changed (+1351 additions, -919 deletions)

View changed files

📝 app/Actions/Comment.php (+2 -0)
📝 app/Actions/Tag.php (+3 -0)
📝 app/Application.php (+6 -7)
📝 app/Auth/Role.php (+3 -0)
📝 app/Auth/User.php (+2 -0)
📝 app/Config/api.php (+0 -0)
📝 app/Config/app.php (+7 -4)
📝 app/Config/auth.php (+13 -2)
📝 app/Config/broadcasting.php (+14 -6)
➕ app/Config/cache.php (+104 -0)
📝 app/Config/database.php (+0 -0)
📝 app/Config/debugbar.php (+0 -0)
📝 app/Config/dompdf.php (+0 -0)
📝 app/Config/filesystems.php (+10 -3)
📝 app/Config/hashing.php (+0 -0)
📝 app/Config/logging.php (+5 -8)
📝 app/Config/mail.php (+0 -0)
📝 app/Config/oidc.php (+0 -0)
📝 app/Config/queue.php (+14 -10)
📝 app/Config/saml2.php (+0 -0)

...and 66 more files

📄 Description

This pull request includes the changes for upgrading to Laravel 8.x. Feel free to commit any additional changes to the shift-51221 branch.

Before merging, you need to:

Checkout the shift-51221 branch
Review all pull request comments for additional changes
Run composer update (if the scripts fail, add --no-scripts)
Clear any config, route, or view cache
Thoroughly test your application (no tests?, no CI?)

If you need help with your upgrade, check out the Human Shifts. You may also join the Shifty Coders Slack workspace to level-up your Laravel skills.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/BookStackApp/BookStack/pull/3021 **Author:** [@ssddanbrown](https://github.com/ssddanbrown) **Created:** 10/28/2021 **Status:** ✅ Merged **Merged:** 10/30/2021 **Merged by:** [@ssddanbrown](https://github.com/ssddanbrown) **Base:** `laravel_8_upgrade` ← **Head:** `shift-51221` --- ### 📝 Commits (10+) - [`5b51039`](https://github.com/BookStackApp/BookStack/commit/5b510398f37e613df3aa7ab398cb669ad743b9bc) Apply Laravel coding style - [`ad2c749`](https://github.com/BookStackApp/BookStack/commit/ad2c74995dab0e14b0b91cbea01f65a274ac4bf0) Shift exception handler - [`6950e6d`](https://github.com/BookStackApp/BookStack/commit/6950e6db3a56d746666d59330ff0c63312884b3c) Shift HTTP kernel and middleware - [`b5db904`](https://github.com/BookStackApp/BookStack/commit/b5db904db93f3eae91de2643d0eba3bcba812e40) Shift service providers - [`eb6a2c7`](https://github.com/BookStackApp/BookStack/commit/eb6a2c728e010000a5d4496ee70953c7fa370738) Convert options array to fluent methods - [`7dbdaca`](https://github.com/BookStackApp/BookStack/commit/7dbdacaaf55cb1cba199ddd9068f843b436c26dd) Shift to class based routes - [`b3898ac`](https://github.com/BookStackApp/BookStack/commit/b3898ac1fedd694edcd26b25b32c08d5107f5756) Shift console routes - [`01dd2b7`](https://github.com/BookStackApp/BookStack/commit/01dd2b74a8e76c6234b2b58274e6342b0c2fe954) Ignore temporary framework files - [`3cd292a`](https://github.com/BookStackApp/BookStack/commit/3cd292a5c3b6a5a3cf83535aa82f1ef1f19e7109) Shift to class based factories - [`5604965`](https://github.com/BookStackApp/BookStack/commit/5604965154abb9320501b9f0edd87ab7f362baa2) Namespace seeders ### 📊 Changes **86 files changed** (+1351 additions, -919 deletions) <details> <summary>View changed files</summary> 📝 `app/Actions/Comment.php` (+2 -0) 📝 `app/Actions/Tag.php` (+3 -0) 📝 `app/Application.php` (+6 -7) 📝 `app/Auth/Role.php` (+3 -0) 📝 `app/Auth/User.php` (+2 -0) 📝 `app/Config/api.php` (+0 -0) 📝 `app/Config/app.php` (+7 -4) 📝 `app/Config/auth.php` (+13 -2) 📝 `app/Config/broadcasting.php` (+14 -6) ➕ `app/Config/cache.php` (+104 -0) 📝 `app/Config/database.php` (+0 -0) 📝 `app/Config/debugbar.php` (+0 -0) 📝 `app/Config/dompdf.php` (+0 -0) 📝 `app/Config/filesystems.php` (+10 -3) 📝 `app/Config/hashing.php` (+0 -0) 📝 `app/Config/logging.php` (+5 -8) 📝 `app/Config/mail.php` (+0 -0) 📝 `app/Config/oidc.php` (+0 -0) 📝 `app/Config/queue.php` (+14 -10) 📝 `app/Config/saml2.php` (+0 -0) _...and 66 more files_ </details> ### 📄 Description This pull request includes the changes for upgrading to Laravel 8.x. Feel free to commit any additional changes to the `shift-51221` branch. **Before merging**, you need to: - Checkout the `shift-51221` branch - Review **all** pull request comments for additional changes - Run `composer update` (if the scripts fail, add `--no-scripts`) - Clear any config, route, or view cache - Thoroughly test your application ([no tests?](https://laravelshift.com/laravel-test-generator), [no CI?](https://laravelshift.com/ci-generator)) If you need help with your upgrade, check out the [Human Shifts](https://laravelshift.com/human-shifts). You may also join the [Shifty Coders](https://laravelshift.com/shifty-coders) Slack workspace to level-up your Laravel skills. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

OVERLORD added the pull-request label 2026-02-05 10:24:51 +03:00

OVERLORD closed this issue

2026-02-05 10:24:53 +03:00

Sign in to join this conversation.

Branches Tags

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: starred/BookStack#6115