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

Possibilité d'accéder à un chapitre après sa suppression #1519

Closed
pierre-24 opened this issue Sep 21, 2014 · 9 comments

Comments

Projects
None yet
8 participants
@pierre-24
Copy link
Member

commented Sep 21, 2014

Comme indiqué par Luthaf sur le forum :

  1. Dans un onglet de navigateur A, créer un big tuto avec un chapitre.
  2. Dans un onglet B, accéder à ce chapitre et le supprimer.
  3. Revenir sur l'onglet A et recharger la page : le chapitre est toujours accessible.

Cause possible : à priori, je pense que la cause est à chercher dans maj_repo_chapter() ou modify_chapter(), avec préférence pour cette dernière option. En particulier, regarder pour quoi sur cette ligne là, il est écrit "maj" et non "del" comme maj_repo_chapter() le voudrait.

@artragis

This comment has been minimized.

Copy link
Member

commented Sep 22, 2014

je ne pense pas que ça soit à cause de ça. En effet le "maj repo part" permet de mettre à jour la partie après qu'on a supprimé le chapitre qui lui était associé.

A mon humble avis, il y a pas mal de chose à changer pour le coup parce que le "maj repo..." est à changer de base dans la ZEP12. Et quand on commencera à développer cette fonctionnalité, on utilisera les conventions django pour recoder ces fonctions plutôt que des fonctions spaghetti comme on l'a actuellement.

@pierre-24

This comment has been minimized.

Copy link
Member Author

commented Sep 22, 2014

je ne pense pas que ça soit à cause de ça. En effet le "maj repo part" permet de mettre à jour la partie après qu'on a supprimé le chapitre qui lui était associé.

Pas faux. Faudrait voir alors ce qui se passe dans le manifest, parce que si le chapitre apparaît quand même, c'est qu'a un endroit, les tests ne se font pas (le fait que les fichiers markdowns correspondant existent toujours est l'objet d'une autre issue ... Qu'il faudrait que je finisse).

A mon humble avis, il y a pas mal de chose à changer pour le coup parce que le "maj repo..." est à changer de base dans la ZEP12. Et quand on commencera à développer cette fonctionnalité, on utilisera les conventions django pour recoder ces fonctions plutôt que des fonctions spaghetti comme on l'a actuellement.

On est bien d'accord. Maintenant, la question est de savoir si le bug est "urgent" ou si on peut attendre l'implémentation de la ZEP-12 (miam) pour le régler.

@firm1

This comment has been minimized.

Copy link
Contributor

commented Sep 22, 2014

Etant donné la nature du bug, je ne pense pas qu'il soit urgentissime. Cependant, si la résolution du bug est triviale (je ne l'ai pas encore regardé de prêt donc je ne peux pas dire) autant le résoudre, au contraire si ça doit prendre plus de 3h (dev+tests+qa), autant passer ces heures sur la ZEP-12.

@SpaceFox SpaceFox added this to the "Futur proche" (v1.x) milestone Sep 24, 2014

@Luthaf

This comment has been minimized.

Copy link

commented Oct 8, 2014

Alors concernant le tuto qui m'a fait découvrir ce problème, après téléchargement, édition à la main du manifest.json et suppression des fichiers puis upload, le problème est réparé.

C'est une manière comme une autre de fixer les tutos cassés.

@Eskimon Eskimon added the zep-12 label Jan 16, 2015

@Eskimon

This comment has been minimized.

Copy link
Member

commented Jan 16, 2015

Au point ou on en est, je tag en zep-12 ;)

@artragis

This comment has been minimized.

Copy link
Member

commented Jan 16, 2015

^^
Le 16/01/2015 12:11, Eskimon a écrit :

Au point ou on en est, je tag en zep-12 ;)


Reply to this email directly or view it on GitHub
#1519 (comment).

@Eskimon

This comment has been minimized.

Copy link
Member

commented Apr 16, 2015

Je met ici un mot cle : ZEP-12 (pour me rappeler de revenir ici) . @pierre-24 @artragis vous confirmez que ca sera dedans ?

@artragis

This comment has been minimized.

Copy link
Member

commented Apr 16, 2015

Need test, mais il me semble que @pierre-24 https://github.com/pierre-24
a déjà intégré ça dans les TU de base.

2015-04-16 14:28 GMT+02:00 Eskimon notifications@github.com:

Je met ici un mot cle : ZEP-12 (pour me rappeler de revenir ici) .
@pierre-24 https://github.com/pierre-24 @artragis
https://github.com/artragis vous confirmez que ca sera dedans ?


Reply to this email directly or view it on GitHub
#1519 (comment)
.

@pierre-24

This comment has been minimized.

Copy link
Member Author

commented Apr 16, 2015

Yep, c'est dans les T.U., donc on devrait être bon là dessus ;)

(need QA, as usual)

@Eskimon Eskimon removed the zep-12 label Apr 21, 2015

@Eskimon Eskimon removed this from the "Futur proche" (v1.x) milestone Apr 21, 2015

@artragis artragis referenced this issue Jul 18, 2015

Closed

liste de tous les tickets #206

23 of 23 tasks complete

@artragis artragis referenced this issue Aug 7, 2015

Merged

ZEP-12 #2956

@Situphen Situphen closed this Sep 16, 2015

@Situphen Situphen added this to the Version de développement milestone Sep 16, 2015

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.