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

Displays information of his account with the API. #2376

Merged
merged 1 commit into from
Feb 24, 2015
Merged

Displays information of his account with the API. #2376

merged 1 commit into from
Feb 24, 2015

Conversation

GerardPaligot
Copy link
Member

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

Affiche les informations de son compte à partir de l'API.

QA :

  • Vérifiez que vous récupérez vos informations à partir de l'URL api/membres/mon_profil/ avec une requête authentifiée.
  • Vérifiez que vous obtenez bien une erreur 401 si vous tentez de faire la requête avec une requête non authentifiée.
  • Vérifiez que la route est générée dans Swagger avec sa documentation.

@firm1
Copy link
Contributor

firm1 commented Feb 24, 2015

Je viens de mettre à disposition cette PR sur le serveur suivant : http://vps137741.ovh.net

Les coordonnées OAuth2 nécessaire pour tester l'authentification sont les suivantes:

CLIENT_ID : 6i!_@xwmnl;ekywdt=6z4CrGn2Y@Z8wIJQNkc;AC
CLIENT SECRET : XWa.Kl1fuGBD?i95WggpPoPL@1;JsAkY8U2FcO0pzEG@fY!D1hwNX!@F4v2g0leVTRaX=KOP3QGn9N65Wsjf0DuOKzKEenCqZ.APnvptAoFO5!kM:q6Adeb!HM4;0jki

La QA est ouverte pour qui veut.

@GerardPaligot
Copy link
Member Author

J'aime ton VPS firm1.

@cgabard
Copy link
Contributor

cgabard commented Feb 24, 2015

J'aurais peut être du temps pour tester ça ce soir, je vous ferais un retour.

@GerardPaligot
Copy link
Member Author

J'ai tenté de générer des tokens :

{
    "error": "invalid_client"
}

@firm1
Copy link
Contributor

firm1 commented Feb 24, 2015

J'ai tenté de générer des tokens :

C'est parce que dans l'admin, tu ne dois pas mettre le nom de l'user mais son pk.

@GerardPaligot
Copy link
Member Author

Ce que je voulais dire, c'est que j'ai tenté de générer les tokens d'authentification avec ton client et qu'il me dit que le client n'est pas valide.

@firm1
Copy link
Contributor

firm1 commented Feb 24, 2015

Ah oui, c'était ma faute, je n'avais pas appuyé sur le bouton "créer". C'est fait (j'ai mis à jour les clés ci-dessus)

@GerardPaligot
Copy link
Member Author

Bon, ça fonctionne mais je ne peux pas me QA. ^^

@SpaceFox
Copy link
Contributor

Authentifié :

{
    "pk": 3,
    "username": "user",
    "show_email": false,
    "email": "",
    "is_active": true,
    "site": "",
    "avatar_url": null,
    "biography": "",
    "sign": "",
    "email_for_answer": false,
    "last_visit": null,
    "date_joined": "2015-02-24T11:39:19"
}

Non authentifié : HTTP 401

{
    "detail": "Authentication credentials were not provided."
}

La route est bien dans Swagger.

Visiblement ça marche, donc je merge :)

@SpaceFox
Copy link
Contributor

Ah non, les erreurs dans Travis ne sont pas celles de d'habitude. Vous pouvez regarder ?

@Eskimon
Copy link
Contributor

Eskimon commented Feb 24, 2015

Ah non, les erreurs dans Travis ne sont pas celles de d'habitude. Vous pouvez regarder ?

C'est pas les PEP-8 qu'on se traine sur la branche de release depuis un moment ? Car les TU sont OK d'apres Travis.

@firm1
Copy link
Contributor

firm1 commented Feb 24, 2015

Je confirme la branche de release a des erreurs de pep8 depuis, donc il ne faut pas s'y fier.

@SpaceFox
Copy link
Contributor

OK, alors je merge.

SpaceFox added a commit that referenced this pull request Feb 24, 2015
Displays information of his account with the API.
@SpaceFox SpaceFox merged commit 67288da into zestedesavoir:release-v1.6 Feb 24, 2015
@GerardPaligot GerardPaligot deleted the feat_2374 branch February 24, 2015 12:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants