Merge pull request #85 from dontcallmedom/fallback-no-pr-title

Add fallback for displaying PR number in digest when title not in API…

Author: dontcallmedom

index.py

@@ -211,6 +211,16 @@ def listify(l):
     newissues = list(filter(isNew, list(filter(isIssue, filtered_events))))
     closedissues = list(filter(isClosed, list(filter(isIssue, filtered_events))))
     newpr = list(filter(isNew, list(filter(isPR, filtered_events))))
+    # as of October 2025, the GitHub Events API doesn't include
+    # title and html_url in PullRequestEvents
+    # as fallback, calculate html_url manually
+    for pr in newpr:
+        html_url = pr["payload"]["pull_request"].get("html_url", None)
+        if not html_url:
+            import sys
+            html_url = pr["payload"]["pull_request"]["url"].replace("/api.", "/").replace("/repos/", "/").replace("/pulls/","/pull/")
+            pr["payload"]["pull_request"]["html_url"] = html_url
+
     mergedpr = list(
         filter(isMerged, list(filter(isClosed, list(filter(isPR, filtered_events)))))
     )

templates/generic/digest

@@ -45,7 +45,7 @@
 * {{name}} (+{{newpr.count}}/-{{mergedpr.count}}/💬{{prcommentscount}})
 {{#newpr.count}}  {{.}} pull requests submitted:
 {{#newpr.list}}
-  - {{{payload.pull_request.title}}} (by {{actor.display_login}})
+  - {{{payload.pull_request.number}}} {{{payload.pull_request.title}}} (by {{actor.display_login}})
     {{{payload.pull_request.html_url}}} {{#payload.pull_request.labels}}[{{name}}] {{/payload.pull_request.labels}}
 {{/newpr.list}}
 

templates/generic/digest.html

@@ -80,7 +80,7 @@ <h3>{{name}} (+{{newissues.count}}/-{{closedissues.count}}/💬{{issuecommentsco
 <h3>{{name}} (+{{newpr.count}}/-{{mergedpr.count}}/💬{{prcommentscount}})</h3>
 {{#newpr.count}}  <p class="new">{{.}} pull requests submitted:</p>
   <ul>{{#newpr.list}}
-  <li>#{{payload.pull_request.number}} <a href="{{payload.pull_request.html_url}}">{{payload.pull_request.title}}</a> (by {{actor.display_login}}) {{#payload.pull_request.labels}}<span class="label" style="background-color: #{{color}}; color: #{{text_color}}">{{name}}</span> {{/payload.pull_request.labels}}</li>
+  <li><a href="{{payload.pull_request.html_url}}">#{{payload.pull_request.number}} {{payload.pull_request.title}}</a> (by {{actor.display_login}}) {{#payload.pull_request.labels}}<span class="label" style="background-color: #{{color}}; color: #{{text_color}}">{{name}}</span> {{/payload.pull_request.labels}}</li>
   {{/newpr.list}}</ul>
 {{/newpr.count}}
 

tests/digest-weekly-repofiltered.msg

@@ -53,21 +53,21 @@ Pull requests
 -------------
 * w3c/webrtc-pc (+8/-1/💬10)
   8 pull requests submitted:
-  - Revert "Respec now uses yarn instead of npm" (by vivienlacourba)
+  - 948 Revert "Respec now uses yarn instead of npm" (by vivienlacourba)
     https://github.com/w3c/webrtc-pc/pull/948 
-  - Splitting transceiver direction into "direction" and "currentDirection". (by taylor-b)
+  - 946 Splitting transceiver direction into "direction" and "currentDirection". (by taylor-b)
     https://github.com/w3c/webrtc-pc/pull/946 
-  - Effect of setting sender.track to null (by aboba)
+  - 944 Effect of setting sender.track to null (by aboba)
     https://github.com/w3c/webrtc-pc/pull/944 
-  - Use of "stopped" in insertDTMF and replaceTrack (by aboba)
+  - 940  (by aboba)
     https://github.com/w3c/webrtc-pc/pull/940 
-  - Removed "stopped" from removeTrack (by aboba)
+  - 939 Removed "stopped" from removeTrack (by aboba)
     https://github.com/w3c/webrtc-pc/pull/939 
-  - What happens when setDirection() is called (by aboba)
+  - 938 What happens when setDirection() is called (by aboba)
     https://github.com/w3c/webrtc-pc/pull/938 
-  - What happens when transceiver.stop() is called (by aboba)
+  - 937 What happens when transceiver.stop() is called (by aboba)
     https://github.com/w3c/webrtc-pc/pull/937 
-  - Remove Issue 1: Current and Pending SDP (by aboba)
+  - 936 Remove Issue 1: Current and Pending SDP (by aboba)
     https://github.com/w3c/webrtc-pc/pull/936 
 
 

tests/digest-weekly.msg

@@ -50,21 +50,21 @@ Pull requests
 -------------
 * w3c/webrtc-pc (+8/-1/💬10)
   8 pull requests submitted:
-  - Revert "Respec now uses yarn instead of npm" (by vivienlacourba)
+  - 948 Revert "Respec now uses yarn instead of npm" (by vivienlacourba)
     https://github.com/w3c/webrtc-pc/pull/948 
-  - Splitting transceiver direction into "direction" and "currentDirection". (by taylor-b)
+  - 946 Splitting transceiver direction into "direction" and "currentDirection". (by taylor-b)
     https://github.com/w3c/webrtc-pc/pull/946 
-  - Effect of setting sender.track to null (by aboba)
+  - 944 Effect of setting sender.track to null (by aboba)
     https://github.com/w3c/webrtc-pc/pull/944 
-  - Use of "stopped" in insertDTMF and replaceTrack (by aboba)
+  - 940  (by aboba)
     https://github.com/w3c/webrtc-pc/pull/940 
-  - Removed "stopped" from removeTrack (by aboba)
+  - 939 Removed "stopped" from removeTrack (by aboba)
     https://github.com/w3c/webrtc-pc/pull/939 
-  - What happens when setDirection() is called (by aboba)
+  - 938 What happens when setDirection() is called (by aboba)
     https://github.com/w3c/webrtc-pc/pull/938 
-  - What happens when transceiver.stop() is called (by aboba)
+  - 937 What happens when transceiver.stop() is called (by aboba)
     https://github.com/w3c/webrtc-pc/pull/937 
-  - Remove Issue 1: Current and Pending SDP (by aboba)
+  - 936 Remove Issue 1: Current and Pending SDP (by aboba)
     https://github.com/w3c/webrtc-pc/pull/936