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

1017 carousel system #1123

Merged
merged 3 commits into from
May 5, 2016

Conversation

willkoua
Copy link
Collaborator

@willkoua willkoua commented May 3, 2016

Q R
Correction de bugs ? [oui
Nouvelle Fonctionnalité ? [oui
Tickets (issues) concernés [Liste de tickets séparés par des virgules]

@@ -28,6 +30,8 @@
*/
class PagesController extends AppController
{
const PATH_CAROUSEL = "../webroot/img/carousel/";

Copy link
Collaborator

Choose a reason for hiding this comment

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

@fxleblanc
Copy link
Collaborator

Étant donné qu'on va utiliser un formulaire, il faudrait ne plus garder les photos dans git.

@@ -28,6 +28,7 @@
*/
class PagesController extends AppController
{

Copy link
Collaborator

Choose a reason for hiding this comment

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

Pas de ligne vide

@fxleblanc
Copy link
Collaborator

Couverture

N'oublie pas de faire un test pour faire remonter la couverture de code

Squash

Une fois que tu auras tout fini, tu pourras faire un squash pour avoir un commit propre au lieu d'une suite de commits semblables

@willkoua willkoua force-pushed the 1017-carousel-system branch 4 times, most recently from 39e9e90 to 8d00f0e Compare May 5, 2016 18:46
correction du code du module carousel selon les normes #2

correction du code du module carousel selon les normes #3

refactorisation du code pour respecter les standards #1

refactorisation du code pour respecter les standards #2

refactorisation du code pour respecter les standards #3

refactorisation du code pour respecter les standards MaisonLogicielLibre#4

refactorisation du code pour respecter les standards MaisonLogicielLibre#5

correction des directives

refactorisation du code pour respecter les standards MaisonLogicielLibre#6
@willkoua willkoua merged commit ab9d2fb into MaisonLogicielLibre:develop May 5, 2016
@willkoua willkoua deleted the 1017-carousel-system branch May 12, 2016 13:46
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.

None yet

2 participants