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

Ajouté de la sécurité dans la config de base (même si noté DEPRECATED) #27

Open
wants to merge 3 commits into
base: dev
Choose a base branch
from

Conversation

piferrari
Copy link
Contributor

Ajout du controller Material and Co.
Divers retouche

Ajout du controller Material and Co.
Divers retouche
$this->load->view('templates/header', ['title' => lang('pf_home')]);
$this->load->view('pages/domain/index', ['domains' => $domains]);
$this->load->view('templates/header', array('title' => lang('pf_home')));
$this->load->view('pages/domain/index', array('domains' => $domains));
Copy link
Member

@greut greut Mar 21, 2017

Choose a reason for hiding this comment

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

La syntaxe [] date de PHP 5.4, c'est dommage de s'en priver non?

Copy link
Member

Choose a reason for hiding this comment

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

Ah, au temps pour moi, j'ai cru qu'elle était là depuis PHP7... J'ai confondu.

Nous sommes encore en attente de savoir quelle version le service info va nous refourguer. Je m'attends à une réponse du genre "PHP6, ça vous va ?" 👼

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Effectivement. On en avait parlé avec Steeve et on était arrivé à la conclusion que notre version PHP de prod. était trop vieille mais je viens de vérifier et oui, on peut (et on va) l'utiliser :)

Ajout de la librairie visjs
Clônage partiel de boilerplate pour html/css/js
Ajout de la librairie visjs
Clônage partiel de boilerplate pour html/css/js
Affichage et gestion partiel des tâches avec visjs
Copy link
Member

@SteeveDroz SteeveDroz left a comment

Choose a reason for hiding this comment

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

Merci de mettre la version 3.2.1 de jQuery, comme discuté.

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