[PR #8] [CLOSED] Create odin.php config file and add login_email setting #41

New issue

Closed

opened 2026-02-27 09:12:43 +03:00 by kerem · 0 comments

kerem commented

2026-02-27 09:12:43 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/maelstrom-cms/odin/pull/8
Author: @fab120
Created: 10/6/2019
Status: ❌ Closed

Base: master ← Head: odin-config-file

📝 Commits (1)

9645e3b Create odin.php config file and add login_email setting

📊 Changes

2 files changed (+17 additions, -1 deletions)

View changed files

➕ config/odin.php (+16 -0)
📝 resources/views/auth/login.blade.php (+1 -1)

📄 Description

If you cache app configs with artisan config:cache the .env file is not parsed anymore and that cause env('LOGIN_EMAIL') on auth/login.blade.php to return a null string instead of the correct value.

To solve this problem, this pr creates a odin.php config file and moves into it the LOGIN_EMAIL setting.

_{🔄 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/maelstrom-cms/odin/pull/8 **Author:** [@fab120](https://github.com/fab120) **Created:** 10/6/2019 **Status:** ❌ Closed **Base:** `master` ← **Head:** `odin-config-file` --- ### 📝 Commits (1) - [`9645e3b`](https://github.com/maelstrom-cms/odin/commit/9645e3b88761f5e03785bd516d03f1aa44d1fcf0) Create odin.php config file and add login_email setting ### 📊 Changes **2 files changed** (+17 additions, -1 deletions) <details> <summary>View changed files</summary> ➕ `config/odin.php` (+16 -0) 📝 `resources/views/auth/login.blade.php` (+1 -1) </details> ### 📄 Description If you cache app configs with `artisan config:cache` the `.env` file is not parsed anymore and that cause `env('LOGIN_EMAIL')` on `auth/login.blade.php` to return a null string instead of the correct value. To solve this problem, this pr creates a `odin.php` config file and moves into it the `LOGIN_EMAIL` setting. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>