Closed Endyl closed 6 years ago
Úgy néz ki, hogy kell is saját settings, de összevonva a src/.env - el. Egyelőre asszem megoldom másolással (post-install), ha valakinek lesz jobb ötlete, akkor code review-n, egyebütt jelezze plz. Várjuk meg #2 -t, valószínű ott több minden is megoldódik.
@Pepita73 Ha nem túl nagy a különbség, akkor nem lenne jobb környezeti változóba betenni, aztán onnan berántani az éles szerveren? Persze nem nehéz a fájl csere vagy append sem, csak egy ötlet volt...
@inf3rno see: https://github.com/Pepita73/webproghu_dev/pull/14 így db config már környezeti változóbol jön, hogy ezt deploy során / után honnan seteljük, azt rád bíznám. :)
@Endyl szerintem ez megoldva #14 - ben, ha egyetértesz, akkor ez is zárható.
@Pepita73 Gondolom én nem tudom követni, hogy mi történik, csak annyi tűnt fel a commit-ból, hogy a settings fájlban kétszer van a databases.default.default, és a getenv-es változatot felülírja a második. https://github.com/Pepita73/webproghu_dev/pull/14/commits/20b5c86fa09a9c84f7341875ea9b296c68bbdd4b#diff-9480fd8c5120147be5a14e828e1218b9R91 https://github.com/Pepita73/webproghu_dev/pull/14/commits/20b5c86fa09a9c84f7341875ea9b296c68bbdd4b#diff-9480fd8c5120147be5a14e828e1218b9R798
Biztos, hogy ez volt a cél?
@inf3rno nem, ezt így elqrtam, rosszkor másoltam ki a fájlt. Erre PR-en mehetett volna rögtön change request, mert ez így tuti rossz. Gyanítom, hogy a webes telepítés során írja hozzá a Drupal (nem tudom, miért).
Nice catch, köszi!
@Pepita73 Legalább valami hasznom van a szájaláson kívül. :D
@inf3rno Egyébként a második settings még onnan maradt benne, amikor csak a LINKS volt meg és weben meg kellett adnom a db kapcsolatot. Akkor írja a Drupal a fájl végére.
Tehát amikor megcsináltam a .env-es játékot, úgy csináltam a PR-t, hogy nem is tudtam róla, hogy nem az működik, hanem az előző webes setup. :-O
Most már a src/web/sites/default/settings.php
végén sincs ott, pont azért, mert átlépi a webes telepítés során a beállítását.
@Endyl szerintem ez zárható, ha megfelelőnek látod #14 - et.
Belinkelem ezt is #9 -be, hogy meglegyen milyen env változók kellenek majd.
Szerintem ezen már nincs megválaszolandó kérdés, és szerintem zárható #14 megoldja. @Endyl , te issue-d... :)
Megoldva #14-ben.
Véleményem szerint a
settings.php
publikus és a fejlesztői rendszerre vonatkozó részét nyugodtan kezelhetjük gittel (ezeket jelenleg ignorálja az alap composeres projekt.gitignore
-ja). Az éles rendszerre vonatkozó beállításokat (adatbázis hozzáférés, one time loginokhoz használt salt, stb.) valahogy a deploy megoldásnak kéne vagy külön fájlként vagy asettings.php
végéhez hozzáadva kitennie az éles rendszerre. (#12)Ha #2 megoldódik, akkor azt is hozzá lehet venni a fejlesztői beállításokhoz.