search

Pepita73 / webproghu_dev

Webprog.hu apache-php7.2, Drupal 8.5.5
1 stars 1 forks source link

Adatbázis IP/host #14

Closed Pepita73 closed 6 years ago

Pepita73 commented 6 years ago

2 fixed

6 eddig vonatkozó része fixed (ha lesz még valami ilyen gond, szóljatok)

Related: #12 , #13 , #11

plz teszteljétek is, nehogy valamit benézzek.

Pepita73 commented 6 years ago

Még ne legyen review, találtam egy jó helloworld-öt a networkre, holnap még rá nézek.

Endyl commented 6 years ago

Ha slynak azt mondtuk, hogy csak az ip probléma legyen egy PR-ben, akkor ide is elég lenne a user defined network, amiről tutorialt is találtál :)

Egyébként a többi részére vonatkozóan:

  1. src/config/sync-be szerintem nem kell .htaccess, mert a docrooton (src/web) kívül van
  2. a settings.php-t szerintem felesleges src/config-ból másolgatni; legyen nyugodtan a helyén a src/web/sites/default-ban, csak az src/.gitignore-hoz hozzá kell adni a 12. sor után egy !/web/sites/default/settings.php-t
  3. az .env.dev szintén nyugodtan lehet src/-ben, nem kell a src/config-ban tárolni

Ezekből a 2. és a 3. egy külön PR-ben a composer.json vonatkozó módosításaival első ránézésre megoldaná #13-at.

Pepita73 commented 6 years ago

network-hö kell a settings, ahhoz meg célszerű a .env. :)

  1. Nem én tettem oda, hanem a Drupal, mivel nincs ignore-ban, commitáltam
  2. \DrupalProject\composer\ScriptHandler::createRequiredFiles valóban csak akkor másolja, ha nem létezik (erre edig nem figyeltem), viszont Drupal is írhat hozzá. OK, legyen így
  3. settings mellé került, mert egymáshoz van közük, így már (2) kiveszem.
Pepita73 commented 6 years ago

Részemről kész.