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

Modification de la documentation #2451

Merged
merged 7 commits into from
Apr 3, 2015
Merged

Conversation

pierre-24
Copy link
Member

Ça, là, c'est la PR pour laquelle tout le monde va me détester, qui va prendre un mois à ce faire QA et qui va générer une tonne de discutions, de rebases et ainsi de suite ;)

Q R
Correction de bugs ? non
Nouvelle Fonctionnalité ? oui
Tickets (issues) concernés Zest'Meeting du 10/04/2015

But

Note de QA

Se placer dans le dossier de documentation et régénérer celle-ci (nécessite SPHINX, qui est installé comme dépendance du back):

cd doc/
make clean
make html

Vérifier (et j'insiste, parce que je suis spécialiste) que tout est cohérent et qu'il n'y a pas de fautes d'orthographe. Ne pas hésiter à me faire part de vos suggestions.

@landscape-bot
Copy link

Code Health
Repository health decreased by 0.03% when pulling 7eddf9f on pierre-24:redoc_things into afa9467 on zestedesavoir:dev.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 79.26% when pulling 7eddf9f on pierre-24:redoc_things into afa9467 on zestedesavoir:dev.

│   └── ...
├── member/ # module des membres
│   └── ...
├── mp/ # module des MPs
Copy link
Member

Choose a reason for hiding this comment

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

Si on met un commentaire, autant être explicite et mettre "messages privés", non ? :)

Copy link
Member Author

Choose a reason for hiding this comment

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

oui :)

@Situphen
Copy link
Member

Merci énormément pour cette PR ! :)

Le *front-end*
==============

Le terme *front-end* désigne la partie du code associée à l'affichage des données issues du `back-end <back-end.html>`__. Il s'agit donc de la partie du code ayant attrait au design et à l'affichage, écrite en HTML (en employant le `langage de template de Django <https://docs.djangoproject.com/en/dev/topics/templates/>`__), à l'aide de SASS pour les feuilles de style et de Javascript (au travers de Node.js).
Copy link
Member

Choose a reason for hiding this comment

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

ça serait peut être intéressant de rappeler que le frontend s'occupe aussi de l'ergonomie, de la réactivité et de l'expérience utilisateur.

Copy link
Member

Choose a reason for hiding this comment

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

Il s'agit donc de la partie du code ayant attrait au design et à l'affichage, écrite en HTML (en employant le langage de template de Django <https://docs.djangoproject.com/en/dev/topics/templates/>__), à l'aide de SASS pour les feuilles de style et de Javascript (au travers de Node.js).

La phrase est complexe je trouve. On peut pas la tourner comme ça :

Il s'agit donc de la partie du code définissant le design et l'affichage. Sa mise en place est basée sur trois langages :

  • Le HTML aidé du langage de template de Django <https://docs.djangoproject.com/en/dev/topics/templates/>__)

  • SASS pour les feuilles de style

  • Javascript pour les intéraction.

    NodeJS, son gestionnaire de packet (npm) et Gulp sont utilisés pour générer le code finale minifié et cohérent.

@artragis
Copy link
Member

Globalement c'est tout bon. j'ai fait deux petites suggestions mais rien de bien grave.

@pierre-24
Copy link
Member Author

Ok, j'ai tenu compte de vos remarques ;)

@landscape-bot
Copy link

Code Health
Repository health decreased by 0.03% when pulling bad9ee9 on pierre-24:redoc_things into afa9467 on zestedesavoir:dev.

@pierre-24
Copy link
Member Author

@Situphen : fait :)

@firm1
Copy link
Contributor

firm1 commented Apr 2, 2015

Pour information, parce que c'est pas super lisible sur le diff, j'ai déployé cette branche à l'adresse suivante : http://vps137741.ovh.net:2451/

Le lien de consultation de la doc est celui-ci : http://vps137741.ovh.net:2451/doc/

@Eskimon
Copy link
Contributor

Eskimon commented Apr 2, 2015

Ca charge ca charge... Marche pas j'ai l'impression... (le port est le même que la PR, serait-ce la mise en oeuvre de ce a quoi que je pense ? :) )

@pierre-24
Copy link
Member Author

Pour information, parce que c'est pas super lisible sur le diff, j'ai déployé cette branche à l'adresse suivante : http://vps137741.ovh.net:2451/

Le lien de consultation de la doc est celui-ci : http://vps137741.ovh.net:2451/doc/

Toi, t'as gagné une boite de chocolat ... Ou presque. Je viens de changer la branche au moment ou t'as eu l'idée ;)

@pierre-24
Copy link
Member Author

@Eskimon : http://vps137741.ovh.net:2451/doc/ ? Ça marche nickel chez moi

@firm1
Copy link
Contributor

firm1 commented Apr 2, 2015

Idem, ça marche chez moi aussi.

@Eskimon
Copy link
Contributor

Eskimon commented Apr 2, 2015

Bon... je vais ressayer a domicile on verra bien

@coveralls
Copy link

Coverage Status

Coverage increased (+0.01%) to 79.27% when pulling 3507e44 on pierre-24:redoc_things into afa9467 on zestedesavoir:dev.

@pierre-24
Copy link
Member Author

Chut, coveralls, cette PR ne peut pas faire chuter la couverture à ce point !!

@landscape-bot
Copy link

Code Health
Repository health decreased by 0.03% when pulling 3507e44 on pierre-24:redoc_things into afa9467 on zestedesavoir:dev.

@firm1
Copy link
Contributor

firm1 commented Apr 2, 2015

Chut, coveralls, cette PR ne peut pas faire chuter la couverture à ce point !!

Mauvaise langue, il est passé à 79% :-°

Fichiers principaux
-------------------

Django étant basé sur une architecture de type Modèle-Vue-Controleur, on retrouve les modèles dans le fichier ``models.py`` et les vues associées à celles-ci dans ``views.py``. Ces dernières peuvent employer des classes formulaires qui sont définis dans ``forms.py``. Les URLs associées au module et permetant d'accéder aux vues sont définies dans ``urls.py``. On retrouve finalement des vues spécifiques associées aux fils RSS et Atom dans ``feeds.py``.
Copy link
Contributor

Choose a reason for hiding this comment

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

une architecture de type Modèle-Vue-Controleur

Avec Django c'est plutot "Modèle Vue Template"

Copy link
Member Author

Choose a reason for hiding this comment

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

Je suis d'accord avec toi, mais j'associerai les templates aux Vues (le contrôleur, c'est Django)

@Eskimon
Copy link
Contributor

Eskimon commented Apr 2, 2015

Yep ca marche a la maison... Et je viens d’innocemment tester une autre PR en changeant le port... @firm1 est un BG !!

@firm1
Copy link
Contributor

firm1 commented Apr 2, 2015

Et je viens d’innocemment tester une autre PR en changeant le port... @firm1 est un BG !!

Attention, ça ne fonctionne pas encore à 100% mon serveur étant relativement petit, je ne peux pas deployer plus de 2 PRs pour le moment

@SpaceFox
Copy link
Contributor

SpaceFox commented Apr 2, 2015

@firm1 Tu pourras nous fournir le script et le dimensionnement de ton serveur pour le jour où on parlera sérieusement de faire tourner tout ça sur des serveurs de l'association ?

@firm1
Copy link
Contributor

firm1 commented Apr 2, 2015

No problem, le script est déjà opensource mais ce n'est pas encore le moment du l'utiliser.

├── middlewares/ # codes provenant de sources externes
│   └── ...

On retrouve également dans ce dossier les quelques fichiers suivants, nécéssaires à la configuration et au fonctionnement de Django :
Copy link
Member

Choose a reason for hiding this comment

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

"nécessaires" et pas "nécéssaires" (présent ailleurs dans la documentation)

@artragis
Copy link
Member

artragis commented Apr 3, 2015

j'ai fait une pr sur le dépôt de @pierre-24 pour corriger l'orthographe pierre-24#1

@pierre-24
Copy link
Member Author

Merci @artragis, je merge ça de suite :)

Francois Dambrine and others added 2 commits April 3, 2015 14:05
@pierre-24
Copy link
Member Author

Voilà :-)

@coveralls
Copy link

Coverage Status

Coverage increased (+0.05%) to 79.3% when pulling 1d7a79f on pierre-24:redoc_things into afa9467 on zestedesavoir:dev.

@artragis
Copy link
Member

artragis commented Apr 3, 2015

to be merged, @Eskimon ?

(ah ah, tu peux pas répondre "pong" cette fois ci!)

Eskimon added a commit that referenced this pull request Apr 3, 2015
Modification de la documentation
@Eskimon Eskimon merged commit db70edb into zestedesavoir:dev Apr 3, 2015
@Eskimon
Copy link
Contributor

Eskimon commented Apr 3, 2015

C'est un petit pas pour ZdS, mais un grand bond pour sa documentation !

@Eskimon Eskimon added this to the Version 1.8 milestone Apr 3, 2015
@Eskimon Eskimon added the C-Docs Concerne la documentation label Apr 3, 2015
@artragis
Copy link
Member

artragis commented Apr 3, 2015

et la doc de la zep 12 peut commencer ^^

@firm1
Copy link
Contributor

firm1 commented Apr 3, 2015

Et énorme merci à @pierre-24 de s'être tapé tout ça. C'est du bon boulot

@pierre-24
Copy link
Member Author

... L'aventure est pas tout à fait terminée :'(

On dirait que RTD a une erreur d'encodage ? :o

@artragis
Copy link
Member

artragis commented Apr 3, 2015

étrange... chez moi ça marche.

@pierre-24
Copy link
Member Author

Chez moi aussi, bien entendu.

@pierre-24
Copy link
Member Author

(et je viens de checker, j'ai pas mis un fichier en ISO-8859-1 pour faire mon troll, c'est UTF-8 partout. Par contre, il bloque sur la partie "critique", à savoir la documentation de code)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-Docs Concerne la documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

8 participants