Skip to content
This repository has been archived by the owner on Jul 22, 2022. It is now read-only.

Configuration cachée #177

Merged
merged 4 commits into from
May 19, 2016
Merged

Configuration cachée #177

merged 4 commits into from
May 19, 2016

Conversation

prytoegrian
Copy link
Member

@prytoegrian prytoegrian commented May 15, 2016

Statut :

  • terminée

Petite PR de repos entre le planning et les autres sur le feu. Ça me gonflait quelque peu de devoir dégainer le git stash en permanence pour changer de branche de travail, associé au fait d'avoir constamment dans git des fichiers à supprimer mentalement du travail en cours.

En conséquence, j'ai fait en sorte que la configuration du projet ne soit plus versionnée mais cachée. Ça ne change rien pour le dev, le travail d'édition des fichiers est comme avant.

Je n'ai bossé ni sur les fichiers LDAP ni CAS car ce sont des systèmes que je ne connais pas et je ne préfère pas changer un truc que je ne peux pas tester.

Note : comme d'hab, le diff de dbconnect s'est vautré à cause des tabulations, tentez-le à la main.

@prytoegrian prytoegrian self-assigned this May 15, 2016
@prytoegrian prytoegrian added this to the 1.9 milestone May 15, 2016
@Shadok
Copy link
Contributor

Shadok commented May 18, 2016

Je n'ai pas tout compris, mais pour le LDAP comme on en parlait, tu peux t'installer un openldap dans ton VM/ton Docker.
Pour de l'AD, si t'as une ip fixe, on doit pouvoir s'arranger pour que je te crée un compte sur le nôtre et t'autorises à t'y connecter.

dbconnect.php to cfg/dbconnect.php
define_new.php to define.php
* `dbconnect.php` to `cfg/dbconnect.php`
* `define_new.php` to `define.php`
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

c'est plus le cas pour define.php

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah oui tiens pourquoi j'ai fait ça ?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah ok j'ai. define_new.php existe jusqu'à la version 1.7, c'est pour ça que j'ai modifié. Vu qu'on est encore compatible, il vaudrait mieux le laisser

@wouldsmina
Copy link
Member

wouldsmina commented May 19, 2016

vu


Copy (and edit):
dbconnect.php to cfg/dbconnect.php
define_new.php to define.php
* `dbconnect.php` to `cfg/dbconnect.php`
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

cfg/dbconnect_new.php to...

@prytoegrian prytoegrian merged commit 59ffd58 into libertempo:develop May 19, 2016
@prytoegrian prytoegrian deleted the topic/envInFile branch May 19, 2016 21:29
@prytoegrian prytoegrian mentioned this pull request Jun 8, 2016
18 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants