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

Relase 1.7 : le slug dans un mp incorect #2510

Closed
firm1 opened this issue Apr 7, 2015 · 6 comments
Closed

Relase 1.7 : le slug dans un mp incorect #2510

firm1 opened this issue Apr 7, 2015 · 6 comments
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
Milestone
Version 1.7

Comments

@firm1
Copy link
Contributor

firm1 commented Apr 7, 2015

Version : Beta 1.7-RC2

Scénario :

  • J'ai crée un MP avec 1 participant "test"
  • J'ai cliqué sur "Ajouter un membre" et j'ai rajouté "test2"
  • L'url de mon MP est devenu : http://beta.zestedesavoir.com/mp/3101/%3Cbound%20method%20PrivateTopic.slug%20of%20%3CPrivateTopic%3A%20test%3E%3E/messages/

Il y'a donc un petit problème par là.
@firm1 firm1 added S-BUG Corrige un problème C-Back Concerne le back-end Django labels Apr 7, 2015
@Eskimon
Copy link
Contributor

Eskimon commented Apr 7, 2015

Je suppose que ca vient d'ici ? #2424

( @SpaceFox on marque en milestone 1.7 pour s'y retrouver ?)

@GerardPaligot
Copy link
Member

@Eskimon : Ce que tu link, c'est ma PR incomplète pour corriger l'erreur (qui avait déjà été remonté pour d'autres cas).

La correction du bug est extrêmement simple : changer self.object.slug en self.object.slug() ici.

@Eskimon Eskimon added the Facile Bon ticket pour débuter pour rejoindre le développement ! label Apr 7, 2015
@Eskimon
Copy link
Contributor

Eskimon commented Apr 7, 2015

Ok super :) je tag facile du coup :)

@SpaceFox SpaceFox added the Bloquant Ticket qui doit être traité avant la prochaine mise à jour label Apr 7, 2015
@SpaceFox SpaceFox added this to the "Futur proche" (v1.x) milestone Apr 7, 2015
@SpaceFox
Copy link
Contributor

SpaceFox commented Apr 7, 2015

Par contre c'est bloquant si on ne peut plus utiliser les MP.

@GerardPaligot
Copy link
Member

Non, non, on peut utiliser les MPs. Les slugs ne sont pas utilisés dans le back, c'est juste pour faire des URLs jolies ...

@GerardPaligot GerardPaligot removed the Bloquant Ticket qui doit être traité avant la prochaine mise à jour label Apr 7, 2015
@Situphen Situphen mentioned this issue Apr 7, 2015
Merged
Situphen added a commit to Situphen/zds-site that referenced this issue Apr 7, 2015
@Situphen
[Betafix] fix zestedesavoir#2510 - Corrige les slugs des MPs
5b9b667
@Eskimon Eskimon modified the milestones: Version 1.7, "Futur proche" (v1.x) Apr 7, 2015
@Eskimon
Copy link
Contributor

Eskimon commented Apr 7, 2015

Réglé

@Eskimon Eskimon closed this as completed Apr 7, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
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
Milestone
Version 1.7
Development

No branches or pull requests
4 participants
@Eskimon @GerardPaligot @SpaceFox @firm1