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

Simplifie l'ajout et l'édition dans les tutos #5142

Merged
merged 9 commits into from Feb 18, 2019

Conversation

4 participants
@A-312
Copy link
Member

A-312 commented Dec 12, 2018

Fonctionnalité, resolve : #5030

Q/A :

  • Vérifier les liens de création/édition que ça redirige vers la bonne page ;
  • Vérifier la présence de "ajouter machin" dans les listes (le plus souvent il y a un "+" devant. -> Vérifier les liens aussi.
  • Lorsqu'on peut ajouter deux choses (partie ou chapitre). On a un message dans le warning : "le contenu est vide. vous pouvez : a ou b".
  • Vérifier aussi la validation et le contenu online.

J'ai ajouté : can_add_something, j'ai donc remplacé l'ancienne double condition :

{% if can_edit or is_staff %}
   {%  if not version or version == content.sha_draft %}

Par :

{% if can_add_something %}

Principe :

  • Lorsque je propose d'ajouter quelques choses en cliquant sur le bouton, j'utilise le block info et non warning.
  • J'ai uniformisé le bloc warning avec le reste du site.
  • Il n'y a pas "Ce chapitre/partie est actuellement vide." si on peut ajouter un, j'ai préféré mettre le bouton d'ajout sans ce bloc warning pour alléger l'interface.

J'ai hésité :

  • A modifier : Cette section est actuellement vide. et ajouter : , vous pouvez [écrire du texte](#).
@coveralls

This comment has been minimized.

Copy link

coveralls commented Dec 12, 2018

Coverage Status

Coverage decreased (-0.03%) to 74.304% when pulling 7d827d1 on A-312:pachasec into e7c946c on zestedesavoir:dev.

@A-312

This comment has been minimized.

Copy link
Member Author

A-312 commented Dec 12, 2018

  • OHOH ! Je suis travis et je suis pas content !

  • OHOH ! Je suis un avion et je ne comprend pas le robot.

En plus il utilise node 8 au lieu de 10.8.0.

@artragis

This comment has been minimized.

Copy link
Member

artragis commented Dec 13, 2018

./zds/tutorialv2/mixins.py:269:121: E501 line too long (128 > 120 characters)

@A-312

This comment has been minimized.

Copy link
Member Author

A-312 commented Dec 13, 2018

Oh je vois merci

@A-312 A-312 force-pushed the A-312:pachasec branch from 8c68aef to 75d9c35 Dec 13, 2018

@A-312

This comment has been minimized.

Copy link
Member Author

A-312 commented Dec 13, 2018

Je ne sais pas si je devais faire comme ça, dites moi.

@artragis

This comment has been minimized.

Copy link
Member

artragis commented Dec 16, 2018

@A-312 je vais faire un billet zest of dev, d'ici peu, ça te dit d'avoir un chapitre où tu présenteras ton taf?

@A-312

This comment has been minimized.

Copy link
Member Author

A-312 commented Dec 17, 2018

@artragis Pourquoi pas, dit moi quand. ;-)

@artragis

This comment has been minimized.

Copy link
Member

artragis commented Dec 17, 2018

je t'ai mis en rédaction du ZOD 11

@A-312

This comment has been minimized.

Copy link
Member Author

A-312 commented Dec 19, 2018

image
image
image
image
image

Partie vide :

vieille image plus récente :
image

2 rendus sont pas bon :

@A-312

This comment has been minimized.

Copy link
Member Author

A-312 commented Dec 19, 2018

Le rendu des tutos vide aussi. >_< (Une de mes dernières modif de "refactoring" a dû faire planter une condition).

@A-312

This comment has been minimized.

Copy link
Member Author

A-312 commented Dec 19, 2018

Ok j'ai oublié un "content." dans mon refactoring, donc :
image
image
image
image
image

(Je crois que j'ai encore fais sauter un commit que je voulais garder)

@A-312

This comment has been minimized.

Copy link
Member Author

A-312 commented Dec 19, 2018

C'est prêt à Q/A, il faudra surement plusieurs personnes pour Q/A cette PR pour être sûr de ne rien oublier. :)

@A-312

This comment has been minimized.

Copy link
Member Author

A-312 commented Dec 19, 2018

Encore un problème sur les billets :

  • image Il manque le "ajouter" dans la liste du sommaire.
  • image (après ma modif, pour corriger le point au dessus) il est en trop.
@A-312

This comment has been minimized.

Copy link
Member Author

A-312 commented Dec 19, 2018

image
image

Voilà, plus de problème. :)

@artragis artragis added this to In progress in Objectif v28.1 via automation Dec 19, 2018

@A-312

This comment has been minimized.

Copy link
Member Author

A-312 commented Dec 20, 2018

Pour refactoring le code : captureas + {% include ... with member=edit.editor avatar=true %}

@artragis artragis moved this from In progress to Needs review in Objectif v28.1 Dec 26, 2018

@artragis

This comment has been minimized.

Copy link
Member

artragis commented Jan 10, 2019

@A-312 tu peux rebase stp?

@Situphen tu peux mettre cette PR sur la béta pour que les gens fassent quelques retours stp?

@Situphen

This comment has been minimized.

Copy link
Contributor

Situphen commented Jan 10, 2019

@artragis Je le fais dès ce soir :)

@A-312

This comment has been minimized.

Copy link
Member Author

A-312 commented Jan 10, 2019

J'allais rebase, donc plus besoin ?

@Situphen

This comment has been minimized.

Copy link
Contributor

Situphen commented Jan 10, 2019

J'allais rebase, donc plus besoin ?

Non plus besoin :)

@Situphen

This comment has been minimized.

Copy link
Contributor

Situphen commented Jan 10, 2019

C'est sur la bêta !

@artragis

This comment has been minimized.

Copy link
Member

artragis commented Jan 22, 2019

@A-312 tu as pu prendre en compte les retours des auteurs?

@A-312

This comment has been minimized.

Copy link
Member Author

A-312 commented Jan 24, 2019

Soucis interne & matériel, j'essaye de regarder ça se weekend

@A-312 A-312 force-pushed the A-312:pachasec branch from 74930f6 to 09f2e81 Jan 29, 2019

@A-312

This comment has been minimized.

Copy link
Member Author

A-312 commented Jan 29, 2019

En gris :
image
En bleu :
image

@A-312

This comment has been minimized.

Copy link
Member Author

A-312 commented Jan 29, 2019

@artragis Voila ;)

@Situphen

This comment has been minimized.

Copy link
Contributor

Situphen commented Jan 29, 2019

J'aime bien le bouton en gris ! D'ailleurs, je pense que ça rendra mieux si tu sors le bouton du <ul> :)

@A-312

This comment has been minimized.

Copy link
Member Author

A-312 commented Jan 29, 2019

J'ai mis en gris en dernier comme ça on peut reverse facilement sans rebase^^ Par contre pour le lien en dehors du ul/ol, ça demande pas mal de changement, j'ai préféré appliquer un margin-left:-18px;
image

@A-312

This comment has been minimized.

Copy link
Member Author

A-312 commented Feb 18, 2019

Les derniers commits étaient prêt :)

@artragis

This comment has been minimized.

Copy link
Member

artragis commented Feb 18, 2019

Impeccable, je la valide car elle était déjà passée sur la béta. Je ferais le rebase de la PR django demain.

@artragis artragis merged commit 5cbe342 into zestedesavoir:dev Feb 18, 2019

2 checks passed

Codacy/PR Quality Review Up to standards. A positive pull request.
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

Objectif v28.1 automation moved this from Needs review to Done Feb 18, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.