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

Lien erroné dans le MP d'ajout à la rédaction d'un contenu #6177

Closed
philippemilink opened this issue Sep 24, 2021 · 1 comment · Fixed by #6343
Closed

Lien erroné dans le MP d'ajout à la rédaction d'un contenu #6177

philippemilink opened this issue Sep 24, 2021 · 1 comment · Fixed by #6343
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

Comments

@philippemilink
Copy link
Member

Description du bug

Le second lien du MP que l'on reçoit lorsqu'on est ajouté comme auteur à un contenu mène vers une page 404.

Comment reproduire ?

La liste des étapes qui permet de reproduire le bug :

  1. Avec user1, créer un billet
  2. Ajouter user2 comme co-auteur du billet
  3. Se connecter comme user2et consulter le MP reçu correspondant à l'ajout comme auteur au billet
  4. Dans le contenu du MP, le second lien (Il a été ajouté à la liste de vos contenus en rédaction ici.) mène vers une 404.

Comportement attendu

Je ne sais pas vraiment quel lien est attendu ici... La liste des contenus en cours de rédaction ?

@philippemilink philippemilink added S-BUG Corrige un problème C-Back Concerne le back-end Django labels Sep 24, 2021
@Arnaud-D
Copy link
Contributor

Le message envoyé utilise ce template : https://github.com/zestedesavoir/zds-site/blob/dev/templates/tutorialv2/messages/add_author_pm.md

Le problème se situe dans le lien contenu dans le paramètre index. Il est initialisé à cet endroit :

url_index = reverse(self.object.type.lower() + ":find-" + self.object.type.lower(), args=[user.pk])

On voit qu'on passe comme argument l'identifiant de l'utilisateur pour fabriquer l'URL, alors qu'on attend son pseudo, comme spécifié ici :

"voir/<str:username>/",

Il faut donc remplacer user.pk par user.username.

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
Development

Successfully merging a pull request may close this issue.

2 participants