feat: use series-detail view instead of patch-list for series

- Solved TODOs of patch-list view being used to refer to series
- Now we handle series-detail urls in the parser for the Depends-on tag

Signed-off-by: Victor Accarini <[email protected]>

Author: victor-accarini

docs/usage/overview.rst

@@ -154,6 +154,7 @@ or a cover letter or the web URL to a patch or a series:
 
     Depends-on: <[email protected]>
     Depends-on: https://pw.example.com/project/myproject/list?series=1234
+    Depends-on: https://pw.example.com/project/myproject/series/1234
 
 .. note::
 

patchwork/parser.py

@@ -1068,7 +1068,10 @@ def find_series_from_url(url):
         logging.warning('Failed to resolve series or patch URL: %s', url)
         return None
 
-    # TODO: Use the series detail view here.
+    if result.view_name == 'series-detail':
+        return Series.objects.get(pk=result.kwargs['series_id'])
+
+    # Handles series as a patch-list view
     if result.view_name == 'patch-list' and parse_result.query:
         # Parse the query string.
         # This can be replaced with something much friendlier once the

patchwork/templates/patchwork/partials/patch-list.html

@@ -179,7 +179,7 @@
         </td>
         <td id="patch-series:{{patch.id}}">
 {% if patch.series %}
-          <a href="?series={{patch.series.id}}">
+          <a href="{% url 'series-detail' series_id=patch.series.id project_id=project.linkname %}">
             {{ patch.series|truncatechars:100 }}
           </a>
 {% endif %}

patchwork/templates/patchwork/submission.html

@@ -76,7 +76,7 @@ <h1>{{ submission.name }}</h1>
   <tr>
     <th>Series</th>
     <td>
-      <a href="{% url 'patch-list' project_id=project.linkname %}?series={{ submission.series.id }}">
+      <a href="{% url 'series-detail' series_id=submission.series.id project_id=project.linkname %}">
         {{ submission.series.name }}
       </a> |
       <button id="toggle-patch-series">expand</button>

releasenotes/notes/add-series-dependencies-6696458586e795c7.yaml

@@ -13,6 +13,7 @@ features:
     ``Depends-on: <[email protected]>``
     Alternatively, the web URL of the patch or series may be given:
     ``Depends-on: http://patchwork.example.com/project/test/list?series=1111``
+    ``Depends-on: http://patchwork.example.com/project/test/series/1111``
 api:
   - |
     The API version has been updated to v1.4.