Skip to content

[FIX] Reviews tab: refused posting clicks must not hide the row - #20

Merged
haimbj1 merged 1 commit into
mainfrom
fix/decided-needs-posted
Sep 28, 2026
Merged

haimbj1 merged 1 commit into
mainfrom
fix/decided-needs-posted

Conversation

@haimbj1

@haimbj1 haimbj1 commented Sep 28, 2026

Copy link
Copy Markdown
Owner

Why

A posting click stores the decision locally at once, but the server can refuse the post (stale-head guard) and queue a re-review instead. The row then counted as decided and — since #17 — vanished from "open work" although nothing was posted. Hit today on cofhe#988 after two held clicks.

What

  • A local decision marks a row decided only when the post actually landed: the review file's status is a posted state (approved / changes-requested / comments-posted) or GitHub shows my review.
  • The approved filter now keys on the same landed evidence only (GitHub APPROVED or the review file's posted status), not on the local click.

Verified with the CI script-extraction + node --check.

@haimbj1
haimbj1 merged commit e8557f3 into main Sep 28, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant