[PR #614] [MERGED] Update web installer #855

Closed
opened 2026-02-04 18:56:01 +03:00 by OVERLORD · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/pelican-dev/panel/pull/614
Author: @Boy132
Created: 10/8/2024
Status: Merged
Merged: 10/12/2024
Merged by: @notAreYouScared

Base: mainHead: update/web-installer


📝 Commits (5)

  • a324ece update web installer
  • 4cb98b3 make sure we have a user
  • 3315ad6 save SESSION_SECURE_COOKIE as text so it's written correctly to the .env
  • a3064b6 set SESSION_COOKIE so session doesn't expire when changing the app name
  • dcfed4d Allow enter to go to next step

📊 Changes

12 files changed (+235 additions, -168 deletions)

View changed files

📝 .env.example (+2 -0)
📝 app/Console/Commands/Environment/AppSettingsCommand.php (+1 -24)
📝 app/Filament/Pages/Installer/PanelInstaller.php (+73 -52)
📝 app/Filament/Pages/Installer/Steps/AdminUserStep.php (+4 -2)
app/Filament/Pages/Installer/Steps/CompletedStep.php (+34 -0)
📝 app/Filament/Pages/Installer/Steps/DatabaseStep.php (+57 -40)
📝 app/Filament/Pages/Installer/Steps/EnvironmentStep.php (+20 -16)
📝 app/Filament/Pages/Installer/Steps/RedisStep.php (+37 -22)
📝 app/Filament/Pages/Installer/Steps/RequirementsStep.php (+6 -4)
📝 app/Http/Controllers/Auth/LoginController.php (+1 -2)
📝 config/panel.php (+0 -4)
📝 resources/views/filament/pages/installer.blade.php (+0 -2)

📄 Description

  • Removed user check in installer
  • Added back old .env variable for installer
  • Simplified p:environment:setup command even more
  • Fixes auto login
  • The APP_URL is now auto filled
  • The setup work is now done after each step (e.g. writing to .env, running migrations, creating the admin user) rather than at the very end
  • Removed "unsaved changes" alert
  • Improved error handling and reporting
  • New "Setup complete" step that handles the queue and crontab stuff:
    grafik
  • Changed .env.example to set a SESSION_COOKIE. This way the page doesn't expire when changing the app name. (By default, Laravel uses the APP_NAME to generate the name of the cookie)

Supersedes #595


🔄 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/pelican-dev/panel/pull/614 **Author:** [@Boy132](https://github.com/Boy132) **Created:** 10/8/2024 **Status:** ✅ Merged **Merged:** 10/12/2024 **Merged by:** [@notAreYouScared](https://github.com/notAreYouScared) **Base:** `main` ← **Head:** `update/web-installer` --- ### 📝 Commits (5) - [`a324ece`](https://github.com/pelican-dev/panel/commit/a324ece089c14bfc14bb8a3a75828683b81536c4) update web installer - [`4cb98b3`](https://github.com/pelican-dev/panel/commit/4cb98b3654b5aee6fc918dbc91a757ca0889bf87) make sure we have a user - [`3315ad6`](https://github.com/pelican-dev/panel/commit/3315ad6749b6a4ee9d4a1e70dc924534af3d0417) save SESSION_SECURE_COOKIE as text so it's written correctly to the .env - [`a3064b6`](https://github.com/pelican-dev/panel/commit/a3064b61f200450f7b8e4739f42d1b944226c6b4) set `SESSION_COOKIE` so session doesn't expire when changing the app name - [`dcfed4d`](https://github.com/pelican-dev/panel/commit/dcfed4d6cb2337b59350329a5ce24a76f93e7856) Allow enter to go to next step ### 📊 Changes **12 files changed** (+235 additions, -168 deletions) <details> <summary>View changed files</summary> 📝 `.env.example` (+2 -0) 📝 `app/Console/Commands/Environment/AppSettingsCommand.php` (+1 -24) 📝 `app/Filament/Pages/Installer/PanelInstaller.php` (+73 -52) 📝 `app/Filament/Pages/Installer/Steps/AdminUserStep.php` (+4 -2) ➕ `app/Filament/Pages/Installer/Steps/CompletedStep.php` (+34 -0) 📝 `app/Filament/Pages/Installer/Steps/DatabaseStep.php` (+57 -40) 📝 `app/Filament/Pages/Installer/Steps/EnvironmentStep.php` (+20 -16) 📝 `app/Filament/Pages/Installer/Steps/RedisStep.php` (+37 -22) 📝 `app/Filament/Pages/Installer/Steps/RequirementsStep.php` (+6 -4) 📝 `app/Http/Controllers/Auth/LoginController.php` (+1 -2) 📝 `config/panel.php` (+0 -4) 📝 `resources/views/filament/pages/installer.blade.php` (+0 -2) </details> ### 📄 Description - Removed user check in installer - Added back old .env variable for installer - Simplified `p:environment:setup` command even more - Fixes auto login - The `APP_URL` is now auto filled - The setup work is now done after each step (e.g. writing to .env, running migrations, creating the admin user) rather than at the very end - Removed "unsaved changes" alert - Improved error handling and reporting - New "Setup complete" step that handles the queue and crontab stuff: ![grafik](https://github.com/user-attachments/assets/dd5ef06c-6dd9-44e9-9dfd-39f342fff5d5) - Changed `.env.example` to set a `SESSION_COOKIE`. This way the page doesn't expire when changing the app name. [(By default, Laravel uses the `APP_NAME` to generate the name of the cookie)](https://github.com/laravel/framework/blob/11.x/config/session.php#L130-L133) Supersedes #595 --- <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-04 18:56:01 +03:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/panel#855