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

Signaler une faute dans un article #2440

Merged
merged 1 commit into from
Mar 31, 2015
Merged

Signaler une faute dans un article #2440

merged 1 commit into from
Mar 31, 2015

Conversation

DevHugo
Copy link
Contributor

@DevHugo DevHugo commented Mar 18, 2015

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

Permet de signaler une faute dans un article.

QA:

  • Créé et publier un article
    • Vérifier que le bouton « signaler une faute dans un article » n'existe pas si on n'est pas connecté
    • Vérifier que le bouton « signaler une faute dans un article » n'existe pas si on est l'auteur de l'article
  • Cliquer sur le bouton pour signaler une faute dans un article
    • Tenter d'envoyer un message vide (au besoin retirer l'attribut required)
    • Vérifier le message d'erreur est correct
  • Valider et connectez vous sur le compte de l'auteur de l'article, le message doit-être correct.

@landscape-bot
Copy link

Code Health
Code quality remained the same when pulling 730f488 on DevHugo:signaler_erreur_article into 0708f8b on zestedesavoir:dev.

@SpaceFox
Copy link
Contributor

Je ne vois pas de tests associés à cette fonctionnalité. Est-ce possible d'en ajouter ?

@DevHugo
Copy link
Contributor Author

DevHugo commented Mar 18, 2015

Ok, je fais ça dans la journée.

@DevHugo
Copy link
Contributor Author

DevHugo commented Mar 18, 2015

C'est fait

@landscape-bot
Copy link

Code Health
Code quality remained the same when pulling 4f0c2e0 on DevHugo:signaler_erreur_article into 0708f8b on zestedesavoir:dev.

1 similar comment
@landscape-bot
Copy link

Code Health
Code quality remained the same when pulling 4f0c2e0 on DevHugo:signaler_erreur_article into 0708f8b on zestedesavoir:dev.

@landscape-bot
Copy link

Code Health
Code quality remained the same when pulling f693c7e on DevHugo:signaler_erreur_article into 0708f8b on zestedesavoir:dev.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.04%) to 78.49% when pulling f693c7e on DevHugo:signaler_erreur_article into 0708f8b on zestedesavoir:dev.

pierre-24
pierre-24 reviewed Mar 19, 2015
View reviewed changes
zds/article/views.py

# Is the user trying to send PM to himself ?
if request.user in article.authors.all():
messages.error(request, _(u'Impossible d\'envoyer la correction car vous êtes auteur de ce tutoriel!'))
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"tutoriel" ^^

@pierre-24
Copy link
Member

Ok pour la QA, à part la petite faute que j'ai signalé. Quand c'est corrigé, à merger :)

(en même temps, je reconnais mon style de code, alors bon ^^)

@landscape-bot
Copy link

Code Health
Code quality remained the same when pulling f693c7e on DevHugo:signaler_erreur_article into c827d7e on zestedesavoir:dev.

@DevHugo
Copy link
Contributor Author

DevHugo commented Mar 20, 2015

A ne pas merger. Il faut que j'applique la correction faite ici: https://github.com/zestedesavoir/zds-site/pull/2433/files

@landscape-bot
Copy link

Code Health
Code quality remained the same when pulling 4cb73ae on DevHugo:signaler_erreur_article into eac1b19 on zestedesavoir:dev.

@landscape-bot
Copy link

Code Health
Repository health increased by 0.03% when pulling 017e3a7 on DevHugo:signaler_erreur_article into eac1b19 on zestedesavoir:dev.

@DevHugo
Copy link
Contributor Author

DevHugo commented Mar 20, 2015

Les corrections de https://github.com/zestedesavoir/zds-site/pull/2433/files ont été appliquées, il faudrait faire une QA. Désolé Pierre, j'avais oublié ce cas bizarre.

@pierre-24
Copy link
Member

C'est surtout que les deux PRs sont arrivées en même temps, donc tu pouvais pas savoir. Je te fais ça tantôt :) (ce soir pour vous, je crois)

@firm1
Copy link
Contributor

firm1 commented Mar 20, 2015

Travis ne semble pas heureux :) à cause de la pep

@DevHugo
Copy link
Contributor Author

DevHugo commented Mar 20, 2015

Comme ça, message/typo/article/(pk) ?

@firm1
Copy link
Contributor

firm1 commented Mar 20, 2015

Comme ça, message/typo/article/(pk) ?

Yop

@landscape-bot
Copy link

Code Health
Repository health increased by 0.03% when pulling 17bfbe9 on DevHugo:signaler_erreur_article into afa9467 on zestedesavoir:dev.

@landscape-bot
Copy link

Code Health
Repository health increased by 0.03% when pulling 10a070b on DevHugo:signaler_erreur_article into afa9467 on zestedesavoir:dev.

@coveralls
Copy link

Coverage Status

Coverage decreased (-31.99%) to 47.27% when pulling 10a070b on DevHugo:signaler_erreur_article into afa9467 on zestedesavoir:dev.

@landscape-bot
Copy link

Code Health
Repository health increased by 0.03% when pulling 5fe6df1 on DevHugo:signaler_erreur_article into afa9467 on zestedesavoir:dev.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.01%) to 79.27% when pulling 5fe6df1 on DevHugo:signaler_erreur_article into afa9467 on zestedesavoir:dev.

@landscape-bot
Copy link

Code Health
Repository health increased by 0.03% when pulling e13f8dc on DevHugo:signaler_erreur_article into afa9467 on zestedesavoir:dev.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.01%) to 79.27% when pulling e13f8dc on DevHugo:signaler_erreur_article into afa9467 on zestedesavoir:dev.

@DevHugo
Copy link
Contributor Author

DevHugo commented Mar 21, 2015

Voila les tests sont passé, si y'a quelqu'un de motivé pour qa ! ^^

@pierre-24
Copy link
Member

Rapport de QA: encore OK pour moi :)

@artragis
Copy link
Member

ping @Eskimon

Eskimon added a commit that referenced this pull request Mar 31, 2015
@Eskimon
Merge pull request #2440 from DevHugo/signaler_erreur_article
b5e1972 
Signaler une faute dans un article
@Eskimon Eskimon merged commit b5e1972 into zestedesavoir:dev Mar 31, 2015
@Eskimon
Copy link
Contributor

Eskimon commented Mar 31, 2015

pong @artragis ;)

@Eskimon Eskimon added this to the Version 1.8 milestone Mar 31, 2015
@SpaceFox SpaceFox modified the milestones: Version 1.8, Version 1.7 Apr 2, 2015
@Eskimon Eskimon removed this from the Version 1.7 milestone Apr 16, 2015
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
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
8 participants
@DevHugo @landscape-bot @SpaceFox @coveralls @pierre-24 @firm1 @artragis @Eskimon