Approval count should show on failure status too #11705

Open
opened 2025-11-02 09:45:21 -06:00 by GiteaMirror · 1 comment
Owner

Originally created by @silverwind on GitHub (Sep 23, 2023).

Description

Noticed this on https://gitea.com/gitea/act/pulls:

image

Both PRs have 1 approval, but only the first one shows "1 approval" in the list. Presumably because the second one failed the checks.

Gitea Version

1.21

Can you reproduce the bug on the Gitea demo site?

Not tested but I assume it would.

Originally created by @silverwind on GitHub (Sep 23, 2023). ### Description Noticed this on https://gitea.com/gitea/act/pulls: <img width="680" alt="image" src="https://github.com/go-gitea/gitea/assets/115237/3893e365-6ca8-4345-bf02-17383ebe9eb0"> Both PRs have 1 approval, but only the first one shows "1 approval" in the list. Presumably because the second one failed the checks. ### Gitea Version 1.21 ### Can you reproduce the bug on the Gitea demo site? Not tested but I assume it would.
GiteaMirror added the type/bug label 2025-11-02 09:45:21 -06:00
Author
Owner

@lng2020 commented on GitHub (Sep 26, 2023):

It is not relevant to action, but it's indeed a bug about official field in review table. See(https://gitea.com/gitea/go-sdk/pulls/631)
ksnip_20230926-121636

@lng2020 commented on GitHub (Sep 26, 2023): It is not relevant to action, but it's indeed a bug about `official` field in `review` table. See(https://gitea.com/gitea/go-sdk/pulls/631) ![ksnip_20230926-121636](https://github.com/go-gitea/gitea/assets/70063547/690d229b-df2a-4f7c-a15e-a21040e079e1)
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/gitea#11705