Gracefully mark more than 50 notifications as done

There is a built-in limit, therefore we need to batch the GraphQL call marking 50 notifications as done at a time. Otherwise we would run into this error message, and no notification would get marked done at all: gh: You can only mark up to 50 notifications as done. Signed-off-by: Johannes Schindelin <[email protected]>
paulbarton90 · Apr 9, 2024 · 529576e · 529576e
1 parent 263e556
commit 529576e
Showing 1 changed file with 6 additions and 1 deletion.
diff --git a/gh-clean-notifications b/gh-clean-notifications
@@ -1,7 +1,6 @@
 #!/usr/bin/env bash
 set -e
 
-
 NOTIFICATION_QUERY="query NotificationsQuery(\$first: Int!, \$endCursor: String, \$miscQuery: String) {
   viewer {
     notificationThreads(first: \$first, after: \$endCursor, query: \$miscQuery) {
@@ -157,8 +156,14 @@ process_notifications() {
       echo "Marking as Done: $(echo "${item}" | jq -r .title)"
       echo "                 https://github.com$(echo "${item}" | jq -r .path)"
       IDS+=(-f "ids[]=$(echo "${item}" | jq -r .id)")
+      if test ${#IDS[@]} -eq 100
+      then
+        gh api graphql --silent "${IDS[@]}" -f query="${MARK_IDS_AS_DONE_QUERY}"
+	IDS=()
+      fi
     done
 
+    test ${#IDS[@]} -eq 0 ||
     gh api graphql --silent "${IDS[@]}" -f query="${MARK_IDS_AS_DONE_QUERY}"
 
   else