From 422cd2da73c7583a1383bb6f5722295711fd8eb5 Mon Sep 17 00:00:00 2001
From: Mouse Reeve <mousereeve@riseup.net>
Date: Fri, 12 Mar 2021 10:37:52 -0800
Subject: [PATCH] Direct message report action

---
 bookwyrm/templates/moderation/report.html              | 2 +-
 bookwyrm/templates/snippets/status/status_options.html | 2 +-
 bookwyrm/urls.py                                       | 5 ++++-
 3 files changed, 6 insertions(+), 3 deletions(-)

diff --git a/bookwyrm/templates/moderation/report.html b/bookwyrm/templates/moderation/report.html
index ce0a0b3a5..32fdf6d74 100644
--- a/bookwyrm/templates/moderation/report.html
+++ b/bookwyrm/templates/moderation/report.html
@@ -16,7 +16,7 @@
 <div class="block content">
     <h3>{% trans "Actions" %}</h3>
     <div class="field is-grouped">
-        <button class="button">Direct message</button>
+        <a class="button" href="{% url 'direct-messages-user' report.user.username %}">{% trans "Send direct message" %}</a>
         <button class="button">Suspend</button>
     </div>
 
diff --git a/bookwyrm/templates/snippets/status/status_options.html b/bookwyrm/templates/snippets/status/status_options.html
index 3c7100558..6f0ca2e6e 100644
--- a/bookwyrm/templates/snippets/status/status_options.html
+++ b/bookwyrm/templates/snippets/status/status_options.html
@@ -22,7 +22,7 @@
 {% else %}
 {# things you can do to other people's statuses #}
 <li role="menuitem">
-    <a href="/direct-messages/{{ status.user|username }}" class="button is-small is-fullwidth">{% trans "Send direct message" %}</a>
+    <a href="{% url 'direct-messages-user' status.user|username %}" class="button is-small is-fullwidth">{% trans "Send direct message" %}</a>
 </li>
 <li role="menuitem">
     {% include 'snippets/report_button.html' with user=status.user status=status %}
diff --git a/bookwyrm/urls.py b/bookwyrm/urls.py
index 551be1e30..0ad464f41 100644
--- a/bookwyrm/urls.py
+++ b/bookwyrm/urls.py
@@ -75,10 +75,13 @@ urlpatterns = [
     re_path(r"^notifications/?$", views.Notifications.as_view()),
     # feeds
     re_path(r"^(?P<tab>home|local|federated)/?$", views.Feed.as_view()),
-    re_path(r"^direct-messages/?$", views.DirectMessage.as_view()),
+    re_path(
+        r"^direct-messages/?$", views.DirectMessage.as_view(), name="direct-messages"
+    ),
     re_path(
         r"^direct-messages/(?P<username>%s)?$" % regex.username,
         views.DirectMessage.as_view(),
+        name="direct-messages-user",
     ),
     # search
     re_path(r"^search/?$", views.Search.as_view()),