#41 Add link in mail notification to the point added from a new request

Fermé
neodarz veut fusionner 2 commits à partir de neodarz/notify_mail_url vers FFDN/master
neodarz a commenté il y a 7 ans

This fix issue #36.

This fix issue #36.
jocelyn a commenté il y a 7 ans
Propriétaire

Sorry for delay, and thanks for contribution :-).

I don't merge as-is because you should use the Django reverse function to build the URL. Less fragile and simpler than crafting the URL by hand. You may need to use the request.build_absolute function, to help you getting the full URL, including hostname.

Good luck, happy to help if needed :-)

Sorry for delay, and thanks for contribution :-). I don't merge as-is because you should use the Django `reverse` function to build the URL. Less fragile and simpler than crafting the URL by hand. You may need to use the `request.build_absolute` function, to help you getting the full URL, including hostname. Good luck, happy to help if needed :-)
neodarz a commenté il y a 7 ans
Publier

Ok, I look this as soon as possible.

Ok, I look this as soon as possible.
jocelyn a commenté il y a 7 ans
Propriétaire

Superseded by #49

Superseded by #49
Veuillez rouvrir cette Pull Request pour effectuer l'opération de fusion.
Connectez-vous pour rejoindre cette conversation.
Aucun jalon
Pas d'assignataire
2 Participants
Chargement…
Annuler
Enregistrer
Il n'existe pas encore de contenu.