Merge pull request #2042 from bookwyrm-social/progress-updates

Fixes progress updates

bookwyrm/forms/forms.py

@@ -45,7 +45,7 @@ class ReportForm(CustomForm):
 
 class ReadThroughForm(CustomForm):
     def clean(self):
-        """make sure the email isn't in use by a registered user"""
+        """don't let readthroughs end before they start"""
         cleaned_data = super().clean()
         start_date = cleaned_data.get("start_date")
         finish_date = cleaned_data.get("finish_date")

bookwyrm/templates/readthrough/readthrough_modal.html

@@ -17,7 +17,14 @@
 {% endblock %}
 
 {% block modal-form-open %}
-<form name="add-readthrough-{{ readthrough.id }}" action="/create-readthrough" method="post">
+<form
+    name="add-readthrough-{{ readthrough.id }}"
+    {% if readthrough.id %}
+    action="{% url 'edit-readthrough' %}"
+    {% else %}
+    action="{% url 'create-readthrough' %}"
+    {% endif %}
+    method="POST">
 {% endblock %}
 
 {% block modal-body %}

bookwyrm/templates/snippets/reading_modals/progress_update_modal.html

@@ -10,6 +10,7 @@
 <form name="reading-progress-{{ uuid }}" action="{% url 'reading-status-update' book.id %}" method="POST" class="submit-status">
 {% csrf_token %}
 <input type="hidden" name="id" value="{{ readthrough.id }}">
+<input type="hidden" name="start_date" value="{{ readthrough.start_date|date:'Y-m-d' }}">
 {% endblock %}
 
 {% block reading-dates %}