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

Fix 2632: Impossible de remettre un champ de son profil à False #2639

Merged
merged 1 commit into from
May 8, 2015
Merged

Conversation

DevHugo
Copy link
Contributor

@DevHugo DevHugo commented May 5, 2015

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

QA

  • Faire un PUT /api/membres/{pk} et tenter de changer de valeurs les champs suivants: shown_sign, shown_email, hover_or_click, email_for_answer.
  • Faire des requêtes vide et vérifier que le champ à toujours la même valeur
  • Vérifier avec des 0 et des 1

@Situphen Situphen added S-BUG Corrige un problème C-API Concerne une API du site labels May 5, 2015
@DevHugo
Copy link
Contributor Author

DevHugo commented May 5, 2015

Peut-être que @GerardPaligot à une meilleure correction à proposer ?

@landscape-bot
Copy link

Code Health
Code quality remained the same when pulling 4e737fd on DevHugo:fix_2632 into 0f4f32f on zestedesavoir:dev.

1 similar comment
@landscape-bot
Copy link

Code Health
Code quality remained the same when pulling 4e737fd on DevHugo:fix_2632 into 0f4f32f on zestedesavoir:dev.

@GerardPaligot
Copy link
Member

  1. Extra_kwargs indique des valeurs par défaut erroné. Les valeurs par défaut sont les valeurs du profil de l'utilisateur. Raison pour laquelle je n'avais pas fait ça.
  2. Travis plante pour une bonne raison.
  3. Il faut rajouter un TU pour éviter de futurs régressions.

@landscape-bot
Copy link

Code Health
Code quality remained the same when pulling 3d5c7ce on DevHugo:fix_2632 into e06d249 on zestedesavoir:dev.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.03%) to 80.07% when pulling 3d5c7ce on DevHugo:fix_2632 into e06d249 on zestedesavoir:dev.

@landscape-bot
Copy link

Code Health
Code quality remained the same when pulling 6ec1836 on DevHugo:fix_2632 into e06d249 on zestedesavoir:dev.

@landscape-bot
Copy link

Code Health
Code quality remained the same when pulling a99498a on DevHugo:fix_2632 into 4d29e17 on zestedesavoir:dev.

@landscape-bot
Copy link

Code Health
Code quality remained the same when pulling 7388c25 on DevHugo:fix_2632 into 4d29e17 on zestedesavoir:dev.

@pierre-24
Copy link
Member

Rapport de QA: c'est bon pour moi ;)

Je vais tout de même attendre confirmation de @GerardPaligot pour être sur, étant pas expert dans l'API, pour le code.

@GerardPaligot
Copy link
Member

Les tests passant au vert, ta QA étant ok, des TU étant rajoutéz et les modifications n'étant pas spécifique à l'API. Ok pour moi.

GerardPaligot added a commit that referenced this pull request May 8, 2015
Fix 2632: Impossible de remettre un champ de son profil à `False`
@GerardPaligot GerardPaligot merged commit 487c83d into zestedesavoir:dev May 8, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-API Concerne une API du site S-BUG Corrige un problème
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants