[PR #3012] [MERGED] Laravel 6 to Laravel 8 Upgrade #6116

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/3012
Author: @ssddanbrown
Created: 10/27/2021
Status: ✅ Merged
Merged: 11/5/2021
Merged by: @ssddanbrown

Base: master ← Head: laravel_upgrade

📝 Commits (6)

f77236a Laravel 7.x Shift (#3011)
f139cde Laravel 8 shift squash & merge (#3029)
98b23fd Moved from debugbar to clockwork
cccee08 Updated API examples with date format changes
9c680ef Updated php packages, Added php8.1 to GH actions
6f75aa9 Reverted shift change to old migration

📊 Changes

308 files changed (+3331 additions, -2815 deletions)

View changed files

📝 .github/workflows/phpunit.yml (+2 -2)
📝 .github/workflows/test-migrations.yml (+2 -2)
📝 app/Actions/Activity.php (+1 -1)
📝 app/Actions/Comment.php (+2 -0)
📝 app/Actions/Tag.php (+3 -0)
📝 app/Api/ApiToken.php (+1 -1)
📝 app/Api/ApiTokenGuard.php (+2 -2)
📝 app/Auth/Role.php (+6 -3)
📝 app/Auth/SocialAccount.php (+1 -1)
📝 app/Auth/User.php (+5 -3)
📝 app/Config/app.php (+7 -4)
📝 app/Config/auth.php (+13 -2)
📝 app/Config/broadcasting.php (+57 -29)
📝 app/Config/cache.php (+69 -36)
➕ app/Config/clockwork.php (+415 -0)
📝 app/Config/filesystems.php (+10 -3)
📝 app/Config/logging.php (+5 -8)
📝 app/Config/mail.php (+2 -0)
📝 app/Config/queue.php (+14 -10)
📝 app/Entities/Models/Book.php (+3 -0)

...and 80 more files

📄 Description

Using shift to accelerate the upgrade process.

Todo

Laravel 6 to 7 shift
Laravel 7 to 8 shift
Update API examples with new date format.
~~Add back in IDE helper (Removed during L6 to L7 due to dependency mismatches).~~
- Moved to clockwork instead.
Do loads of testing, Some hidden corners:
- Password reset flow/email
- Email confirmation
- Theme system (Logical and Visual)

Things to advise users

Default date serialization has changed which will affect API responses. Think the new behaviour is better but could be considered a breaking change so will need an advisory.

_{🔄 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/3012 **Author:** [@ssddanbrown](https://github.com/ssddanbrown) **Created:** 10/27/2021 **Status:** ✅ Merged **Merged:** 11/5/2021 **Merged by:** [@ssddanbrown](https://github.com/ssddanbrown) **Base:** `master` ← **Head:** `laravel_upgrade` --- ### 📝 Commits (6) - [`f77236a`](https://github.com/BookStackApp/BookStack/commit/f77236aa3832a5a0f050e097c60189b9b6a19969) Laravel 7.x Shift (#3011) - [`f139cde`](https://github.com/BookStackApp/BookStack/commit/f139cded789908efce3ac2ed1be26b947df647db) Laravel 8 shift squash & merge (#3029) - [`98b23fd`](https://github.com/BookStackApp/BookStack/commit/98b23fd7ab98008319f461cafc829cc6f48d1b07) Moved from debugbar to clockwork - [`cccee08`](https://github.com/BookStackApp/BookStack/commit/cccee0808f6a9c67e11da14368fc1ed917b6cd96) Updated API examples with date format changes - [`9c680ef`](https://github.com/BookStackApp/BookStack/commit/9c680efaad1e116be926968f6f329e97d00b4c47) Updated php packages, Added php8.1 to GH actions - [`6f75aa9`](https://github.com/BookStackApp/BookStack/commit/6f75aa9cdcf419ad9046b7d93596204d80fe75b3) Reverted shift change to old migration ### 📊 Changes **308 files changed** (+3331 additions, -2815 deletions) <details> <summary>View changed files</summary> 📝 `.github/workflows/phpunit.yml` (+2 -2) 📝 `.github/workflows/test-migrations.yml` (+2 -2) 📝 `app/Actions/Activity.php` (+1 -1) 📝 `app/Actions/Comment.php` (+2 -0) 📝 `app/Actions/Tag.php` (+3 -0) 📝 `app/Api/ApiToken.php` (+1 -1) 📝 `app/Api/ApiTokenGuard.php` (+2 -2) 📝 `app/Auth/Role.php` (+6 -3) 📝 `app/Auth/SocialAccount.php` (+1 -1) 📝 `app/Auth/User.php` (+5 -3) 📝 `app/Config/app.php` (+7 -4) 📝 `app/Config/auth.php` (+13 -2) 📝 `app/Config/broadcasting.php` (+57 -29) 📝 `app/Config/cache.php` (+69 -36) ➕ `app/Config/clockwork.php` (+415 -0) 📝 `app/Config/filesystems.php` (+10 -3) 📝 `app/Config/logging.php` (+5 -8) 📝 `app/Config/mail.php` (+2 -0) 📝 `app/Config/queue.php` (+14 -10) 📝 `app/Entities/Models/Book.php` (+3 -0) _...and 80 more files_ </details> ### 📄 Description Using shift to accelerate the upgrade process. ### Todo - [x] Laravel 6 to 7 shift - [x] Laravel 7 to 8 shift - [x] Update API examples with new date format. - [ ] ~Add back in IDE helper (Removed during L6 to L7 due to dependency mismatches).~ - Moved to clockwork instead. - [x] Do loads of testing, Some hidden corners: - [x] Password reset flow/email - [x] Email confirmation - [x] Theme system (Logical and Visual) ### Things to advise users - Default [date serialization has changed](https://laravel.com/docs/7.x/upgrade#date-serialization) which will affect API responses. Think the new behaviour is better but could be considered a breaking change so will need an advisory. --- <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:54 +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#6116