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

Corrige deux petites erreur 500 #2634

Merged
merged 2 commits into from
May 10, 2015
Merged

Corrige deux petites erreur 500 #2634

merged 2 commits into from
May 10, 2015

Conversation

Eskimon
Copy link
Contributor

@Eskimon Eskimon commented May 5, 2015

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

Corrige deux petites erreur 500 inoffensives...

QA

  • Faites un sujet de fofo avec deux utilisateurs et quelques messages
  1. Editer un message
    • Sur la page d'edition, alterer l'url pour y mettre des lettres
    • Valider l'URL et constater une 404 au lieu d'une 500
  2. Essayer de citer en ouvrant le lien de citation (donc copier le lien du bouton et ouvrez un nouvel onglet)
    • Sur la page de message, alterer l'url pour y mettre des lettres dans le paramètre cite=
    • Valider l'URL et constater une 404 au lieu d'une 500

@landscape-bot
Copy link

Code Health
Repository health decreased by 1% when pulling 7218978 on Eskimon:fix-2633 into 0f4f32f on zestedesavoir:dev.

@Eskimon
Copy link
Contributor Author

Eskimon commented May 5, 2015

Ah... on dirait que j'ai fait une bêtise dans mes tests... je regarde plus tard.

@DevHugo
Copy link
Contributor

DevHugo commented May 5, 2015

Je viens de regarder le code, je sais ce que c'est mais c'est trés trés bizarre. Tu as oublié de te connecter dans les tests unitaires. La classe de test s’appelle "ForumGuestTests" mais si les tests unitaires sont trés trés bizarre.

Bouge tes tests dans "ForumMemberTests"

@Eskimon
Copy link
Contributor Author

Eskimon commented May 5, 2015

J'ai envie de dire : Oh.Mon.Dieu.

@Situphen Situphen added S-BUG Corrige un problème Facile Bon ticket pour débuter pour rejoindre le développement ! C-Back Concerne le back-end Django labels May 5, 2015
@landscape-bot
Copy link

Code Health
Repository health decreased by 1% when pulling 7218978 on Eskimon:fix-2633 into 4d29e17 on zestedesavoir:dev.

@Eskimon Eskimon changed the title Fix 2633 Corrige deux petites erreur 500 May 7, 2015
@landscape-bot
Copy link

Code Health
Repository health decreased by 1% when pulling 3f3270b on Eskimon:fix-2633 into b7730db on zestedesavoir:dev.

@Eskimon
Copy link
Contributor Author

Eskimon commented May 8, 2015

Bon voila qui est updaté, en local tout passe...

@Situphen
Copy link
Member

Rapport de QA : C'est parfait !

Situphen added a commit that referenced this pull request May 10, 2015
Corrige deux petites erreur 500
@Situphen Situphen merged commit 93d89a1 into zestedesavoir:dev May 10, 2015
@Situphen Situphen added this to the Version de développement milestone May 10, 2015
@Eskimon Eskimon deleted the fix-2633 branch May 10, 2015 20:58
@SpaceFox SpaceFox modified the milestones: Version 15.5.1, Version de développement May 26, 2015
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 Facile Bon ticket pour débuter pour rejoindre le développement ! S-BUG Corrige un problème
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants