From 6975353ee0563d5fb309ccb4fc04b97b5f26cc8e Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=C5=A0t=C4=9Bp=C3=A1n=20Zikmund?= Date: Sun, 13 Sep 2026 22:39:46 +0200 Subject: [PATCH] dev.git: drop MRs you already approved from the GitLab review queue GitLab keeps you on an MR's reviewer list after you approve, so the awaiting-review queue, its count and the bar dot kept every approved but unmerged MR. GitHub's review-requested:@me stops matching once you submit a review, so the GitLab tab was the odd one out. The query now filters on the viewer's own review state and keeps every state except APPROVED. A merge request approved by others but not by you still shows, and one whose approvals were reset by a new push comes back as UNAPPROVED. The count comes from the same field, so the tile and the dot follow without further changes. The tile's link gets the matching not[approved_by_usernames][] filter, so the dashboard it opens lists the same merge requests. Co-Authored-By: Claude Opus 5 --- Panel.qml | 2 +- bin/gitwork | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/Panel.qml b/Panel.qml index fa13c6e..4451527 100644 --- a/Panel.qml +++ b/Panel.qml @@ -365,7 +365,7 @@ Panel { var user = encodeURIComponent(p.username) var mrQueue = origin + "/dashboard/merge_requests?state=opened" var issueQueue = origin + "/dashboard/issues?state=opened" - if (category === "review") return mrQueue + "&reviewer_username=" + user + if (category === "review") return origin + "/dashboard/merge_requests/search?scope=all&state=opened&reviewer_username=" + user + "¬[approved_by_usernames][]=" + user if (category === "assigned") return mrQueue + "&assignee_username=" + user if (category === "assignedIssues") return issueQueue + "&assignee_username=" + user if (category === "authoredIssues") return issueQueue + "&author_username=" + user diff --git a/bin/gitwork b/bin/gitwork index a957e12..f187e22 100755 --- a/bin/gitwork +++ b/bin/gitwork @@ -407,7 +407,7 @@ query { username name webUrl - reviewRequestedMergeRequests(state: opened, first: 50, sort: UPDATED_DESC) { count nodes { ...mr } } + reviewRequestedMergeRequests(state: opened, reviewStates: [UNREVIEWED, REVIEWED, REVIEW_STARTED, REQUESTED_CHANGES, UNAPPROVED], first: 50, sort: UPDATED_DESC) { count nodes { ...mr } } assignedMergeRequests(state: opened, first: 50, sort: UPDATED_DESC) { count nodes { ...mr } } authoredMergeRequests(state: opened, first: 50, sort: UPDATED_DESC) { count nodes { ...mr } } }