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

Étoffement de la doc sur les forums #1765

Merged
merged 3 commits into from
Nov 22, 2014
Merged

Conversation

Axylium
Copy link
Contributor

@Axylium Axylium commented Nov 16, 2014

Q R
 Nouvelles fonctionalité ? Non
 Correction de bug(s)  Non
 Ticket(s) Concerné(s)   Aucun

Cette PR a pour but la rédaction de la doc sur les forums. N'hésitez pas à me faire des retours tant sur les fôtes en tout genre ou si vous avez des critiques sur la syntaxe ou que sais-je encore.

@Axylium
Copy link
Contributor Author

Axylium commented Nov 16, 2014

Si quelqu'un peut m'expliquer comment mettre des images là-dedans. Car j'ai beau suivre les exemples que j'ai trouvé mes images, ne se mettent pas correctement.

@Eskimon Eskimon added C-Back Concerne le back-end Django C-Docs Concerne la documentation and removed C-Back Concerne le back-end Django labels Nov 16, 2014
@Axylium
Copy link
Contributor Author

Axylium commented Nov 22, 2014

Bon, j'ai déjà rédigé une grosse partie.

Si quelqu'un veut relire. Sinon c'est bon pour un merge, je pense !

@gustavi
Copy link
Contributor

gustavi commented Nov 22, 2014

C'est bon pour moi. Il faudrait juste un rebase @Florianboux !

@gustavi
Copy link
Contributor

gustavi commented Nov 22, 2014

C'est pas bon ici, il y a du conflit.

.. figure:: images/deplacement.png
:align: center

>>>>>>> Étoffement de la doc sur les forums
Copy link
Contributor

Choose a reason for hiding this comment

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

Il en reste :/

@Eskimon
Copy link
Contributor

Eskimon commented Nov 22, 2014

Au risque de passer pour un casse pied, je suis contre merger des commits qui laisse apparaitre des marqueurs de conflits... Donc pour moi c'est NO MERGE

@firm1
Copy link
Contributor

firm1 commented Nov 22, 2014

C'est bon pour moi.

Je merge :)

firm1 pushed a commit that referenced this pull request Nov 22, 2014
Étoffement de la doc sur les forums
@firm1 firm1 merged commit 82cb00e into zestedesavoir:dev Nov 22, 2014
@firm1 firm1 removed the En cours label Nov 22, 2014
@Axylium Axylium deleted the doc_forums branch November 22, 2014 17:22
@SpaceFox
Copy link
Contributor

Je comprends pas c'est quoi le bordel ici :

  • Pourquoi ça a été mergé par @firm1 alors que @Eskimon avait demandé à ce que ce ne soit pas mergé ?
  • Où sont les marqueurs de conflits en question ?
  • Pourquoi si ça a été mergé, ça n'est affecté à aucune milestone ?

@Axylium
Copy link
Contributor Author

Axylium commented Nov 22, 2014

Les marqueurs de conflits se trouvent sur ce commit : Axylium@a53aa84

Pour les 2 autres points, je ne sais pas.

@Eskimon
Copy link
Contributor

Eskimon commented Nov 22, 2014

Les marqueurs de conflits sont sur les deux commits précédents. Je trouve personnellement moche de les laisser... Maintenant c'est trop tard, c'est pas grave on en fera pas un fromage vu que le dernier commit répare cela... C'est juste dommage.

@Eskimon Eskimon added this to the Version 1.4 milestone Nov 22, 2014
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.

5 participants