Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Miscaminatas #13

Closed
wants to merge 5 commits into from
Closed

Miscaminatas #13

wants to merge 5 commits into from

Conversation

miscaminatas
Copy link

No description provided.

All supported configuration options based on the DefaultSettings.php file (Never Edit)
DevelopmentSettings.php
Sin configuración S3 para trabajo en "local"
Resto sin cambios
El Contenido del archivo es el de LocalSettings.php (Configuración standard del wiki.partidodelared.org)
No estoy seguro que LocalSetting acepte instrucciones php
Voy a Probar en local
@ghost ghost assigned cristiandouce Sep 14, 2013
@gvilarino
Copy link
Member

@cristiandouce , lo revisás pls?

@cristiandouce
Copy link

Observaciones sobre PR:

  • Los archivos DevelopmentSettings.php y ProductionSettings.php deberían estar en el .gitignore.
  • No termina de ser claro cómo funcionaría el archivo LocalSettings.php.nuevo, o de qué manera lo utiliza la app.
  • Los archivos DevelopmentSettings.php y ProductionSettings.php deberían ser consistentes en cuanto a lo que definen.

Observar cómo se manejan las configuraciones en DemocracyOS:

  1. Definir un archivo DevelopmentSettings.sample.php
  2. Tener un archivo que "cargue" las variables definidas en el entorno; es decir los $_ENV['my-env-var'] y exportarlo en un archivo EnvSettings.php
  3. Manejar en LocalSettings.php:
    1. Dejar en el archivo los "default" settings del proyecto. Lo mínimo para que funcione local.
    2. Si existe DevelopmentSettings.php, requerirlo
    3. Si existe en cambio ProductionSettings.php requerirlo, para deploys fuera de Heroku PaaS.
    4. Requerir luego EnvSettings.php para que haga override sobre los settings ya establecidos. Debería hacer un check de que la variable primero exista, y si lo hace definirla.

Básicamente, tanto production como development deberían entrar en .gitignore. Y archivo LocalSettings.php debería determinar en qué entorno se está corriendo la applicación y cargar las settings correspondientes.

Es mi manera de entenderlo. Por cualquier duda consultar! ⛵

@gvilarino
Copy link
Member

@miscaminatas , deberíamos orientar #11 a lo que dice @cristiandouce .

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants