replace report_issue and resolve_issue endpoints with new "issue_handling" endpoint

CHANGELOG.md

@@ -13,6 +13,11 @@ Types of changes:
 - `Fixed`: for any bug fixes.
 - `Security`: in case of vulnerabilities.
 
+## [x.y.z]
+
+### Changed
+- Issue report and resolve condensed to a single endpoint named `issue_handling`, functionality preserved.
+
 ## [1.12.0]
 
 ### Added

app/host/static/robots.txt

@@ -6,8 +6,7 @@ Disallow: /download_modelfit/
 Disallow: /download_percentiles/
 Disallow: /retrigger_transient/
 Disallow: /reprocess_transient/
-Disallow: /report_issue/
-Disallow: /resolve_issue/
+Disallow: /issue_handling/
 Disallow: /healthz/
 Disallow: /cutout_fits_plot/
 Disallow: /fetch_sed_plot/

app/host/templates/host/processing_status_card.html

@@ -37,15 +37,15 @@ <h4>Processing Status: <span class="badge {{ processing_status_badge_class }}">{
         {% if item.user_warning %}
             <a href="#" type="button" title="Issue reported. Click to clear. (requires authorization)"><span class="report" 
                 {% if is_auth %}
-                onclick="window.open('{% url 'resolve_issue' item.pk %}','_self')"
+                onclick="window.open('{% url 'issue_handling' action='resolve' item_id=item.pk %}','_self')"
                 {% else %}
                 onclick="return false;"
                 {% endif %}
             ><i class="bi bi-exclamation-triangle issue-reported"></i></span></a>
         {% else %}
             <a href="#" type="button" title="No issues reported. Click to report an issue. (requires authorization)"><span class="report" 
                 {% if is_auth %}
-                onclick="window.open('{% url 'report_issue' item.pk %}','_self')"
+                onclick="window.open('{% url 'issue_handling' action='report' item_id=item.pk %}','_self')"
                 {% else %}
                 onclick="return false;"
                 {% endif %}

app/host/urls.py

@@ -50,14 +50,9 @@
         name="retrigger_transient",
     ),
     path(
-        f"""{base_path}report_issue/<item_id>""",
-        views.report_issue,
-        name="report_issue",
-    ),
-    path(
-        f"""{base_path}resolve_issue/<item_id>""",
-        views.resolve_issue,
-        name="resolve_issue",
+        f"""{base_path}issue_handling/<str:action>/<int:item_id>""",
+        views.issue_handling,
+        name="issue_handling",
     ),
     path(f"""{base_path}privacy""", views.privacy_policy, name='privacy'),
     path(f"""{base_path}healthz""", views.healthz, name='healthz'),

app/host/views.py

@@ -741,29 +741,19 @@ def update_home_page_statistics():
     with open(os.path.join(settings.STATIC_ROOT, 'index.html'), 'w') as fp:
         fp.write(html_body)
 
-
-@login_required
-@log_usage_metric()
-def report_issue(request, item_id):
-    item = TaskRegister.objects.get(pk=item_id)
-    item.user_warning = True
-    item.save()
-    return HttpResponseRedirect(
-        reverse_lazy("results", kwargs={"transient_name": item.transient.name})
-    )
-
-
 @login_required
 @log_usage_metric()
-def resolve_issue(request, item_id):
-    item = TaskRegister.objects.get(pk=item_id)
-    item.user_warning = False
+def issue_handling(request, item_id, action):
+    item = get_object_or_404(TaskRegister, pk=item_id)
+    if action == 'report':
+        item.user_warning = True
+    elif action == 'resolve':
+        item.user_warning = False
     item.save()
     return HttpResponseRedirect(
         reverse_lazy("results", kwargs={"transient_name": item.transient.name})
     )
 
-
 # Handler for 403 errors
 def error_view(request, exception, template_name="403.html"):
     return render(request, template_name)