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.

Sign up for GitHub

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

Jump to bottom

[betav16]essaie de corriger le problème des tutos qui disparaissent après un déplacement uniquement s'ils sont publiés. #3373

Merged
merged 3 commits into from
Feb 17, 2016
Merged

[betav16]essaie de corriger le problème des tutos qui disparaissent après un déplacement uniquement s'ils sont publiés. #3373

merged 3 commits into from
Feb 17, 2016

Conversation

artragis
Copy link
Member

Q R
Correction de bugs ? [oui]
Nouvelle Fonctionnalité ? [non]
Tickets (issues) concernés [#3328]

Comme ce bug est difficile à reproduire en local, il serait vraiment sympa de tirer cette PR en preprod.

QA chez soi si vous avez de la chance :

Avant la PR

  1. Depuis cette archive importez le tuto.
  2. Publiez-le.
  3. retournez dans la version hors ligne, opérez un déplacement "descendre" du premier extrait
  4. Une erreur 404 doit être lancée.

Après la PR

  1. Depuis cette archive importez le tuto.
  2. Publiez-le.
  3. retournez dans la version hors ligne, opérez un déplacement "descendre" du premier extrait
  4. Tout doit bien se passer.

QA sur le serveur de preprod si @sandhose a le temps de puller la PR

Parmis vos tutos publiés qui ont dans leur titre soit :

  • un apostrophe (')
  • un point (.)

Opérez les opérations 3 et 4.

@gustavi
Copy link
Contributor

gustavi commented Feb 14, 2016

Je n'arrive pas à reproduire la 404 pre-PR

@artragis
Copy link
Member Author

il faut donc que @sandhose ou @SpaceFox pull la PR sur la preprod

@GerardPaligot
Copy link
Member

Si le bug est identifié, pourquoi n'est-il pas reproductible en local ? Ne suffit-il pas de créer un contenu avec des apostrophes pour reproduire le bug ?

@artragis
Copy link
Member Author

non c'est seulement sur du contenu en prod

Le 16 février 2016 à 11:00, Gérard Paligot notifications@github.com a
écrit :

Si le bug est identifié, pourquoi n'est-il pas reproductible en local ? Ne
suffit-il pas de créer un contenu avec des apostrophes pour reproduire le
bug ?

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

@Situphen Situphen added S-Régression Corrige un problème sur un composant qui fonctionnait auparavant C-Back Concerne le back-end Django Bloquant Ticket qui doit être traité avant la prochaine mise à jour labels Feb 16, 2016
@GerardPaligot
Copy link
Member

@sandhose Tu sais nous déployer ça en bêta ?

@GerardPaligot
Copy link
Member

@SpaceFox Penses-tu que c'est possible de tirer cette branche sur la bêta et de faire une nouvelle synchro prod <-> bêta pour avoir un ensemble de contenus où nous sommes certain qu'ils sont sujets au bug ?

@SpaceFox
Copy link
Contributor

Faut que je trouve le temps de faire ça ce soir.

@GerardPaligot
Copy link
Member

QA : ok sur le serveur de bêta. Merci à @SpaceFox pour la synchro et @artragis pour le bug fix !

GerardPaligot added a commit that referenced this pull request Feb 17, 2016
@GerardPaligot
Merge pull request #3373 from artragis/fix_savage_slug_rename
1506576
@GerardPaligot GerardPaligot merged commit 1506576 into zestedesavoir:release-v16 Feb 17, 2016
@artragis artragis deleted the fix_savage_slug_rename branch April 25, 2021 15:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
Bloquant Ticket qui doit être traité avant la prochaine mise à jour C-Back Concerne le back-end Django S-Régression Corrige un problème sur un composant qui fonctionnait auparavant
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
5 participants
@artragis @gustavi @GerardPaligot @SpaceFox @Situphen