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

correction des liens précendents et suivants dans la chapitres #1117

Closed
wants to merge 1 commit into from

Conversation

firm1
Copy link
Contributor

@firm1 firm1 commented Jul 8, 2014

Q R
Correction de bugs ? oui
Nouvelle Fonctionnalité ? non
Tickets concernés Sur le forum

Note pour QA
Charger un big tuto et utiliser les boutons précédents et suivant pour se ballader. Normalement on ne devrait plus avoir de 404.

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling 5680724 on hotfix-prev-next-tuto into a5d7786 on master.

@artragis
Copy link
Member

artragis commented Jul 9, 2014

Je prends la Q

@artragis
Copy link
Member

artragis commented Jul 9, 2014

Erreur 404 quand le chapitre suivant (ou précédent) est dans la partie suivante (ou précédente).
Cela est du au fait que la clef primaire utilisée pour le suivant/précédent est celle de la partie en cours.

@ArnaudCalmettes ArnaudCalmettes added this to the Version RCx milestone Jul 9, 2014
@Alex-D
Copy link
Contributor

Alex-D commented Jul 9, 2014

Ca viendrait des templates du coup nan ?

EDIT : nop, en fait rien voir, y a des getters que j'utilise, ça ne vient pas de moi du coup.

@firm1
Copy link
Contributor Author

firm1 commented Jul 13, 2014

Je ferme celle-ci car la #1169 le fait aussi.

@firm1 firm1 closed this Jul 13, 2014
@firm1 firm1 deleted the hotfix-prev-next-tuto branch July 13, 2014 17:07
@Alex-D
Copy link
Contributor

Alex-D commented Jul 13, 2014

Tu ne t'es pas trompé ? Les deux PR n'ont rien à voir selon moi.

@Eskimon
Copy link
Contributor

Eskimon commented Jul 14, 2014

N'aurait pas du être fermé:

@firm1
Copy link
Contributor Author

firm1 commented Jul 14, 2014

Je voulais écrire #1158 et j'ai écrit #1169.

C'était donc la #1158 qui incluait le fix.

Concernant la remarque d'artragis je n'arrive pas a reproduire.

@Eskimon
Copy link
Contributor

Eskimon commented Jul 14, 2014

@firm1
Copy link
Contributor Author

firm1 commented Jul 14, 2014

Ah il m'aura fallu plusieurs minute pour comprendre qu'en fait ce bug ne surgit qu'en mode offline.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-Back Concerne le back-end Django S-BUG Corrige un problème
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants