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

Closed
neodarz wants to merge 2 commits from neodarz/notify_mail_url into FFDN/master
neodarz commented 7 years ago

This fix issue #36.

This fix issue #36.
jocelyn commented 7 years ago
Owner

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 commented 7 years ago
Poster

Ok, I look this as soon as possible.

Ok, I look this as soon as possible.
jocelyn commented 7 years ago
Owner

Superseded by #49

Superseded by #49
Please reopen this pull request to perform merge operation.
Sign in to join this conversation.
No Milestone
No assignee
2 Participants
Loading...
Cancel
Save
There is no content yet.